Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
Maddiaa0 committed Feb 7, 2024
1 parent 7580004 commit 4bf841b
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 6 deletions.
1 change: 0 additions & 1 deletion yarn-project/circuit-types/src/logs/function_l2_logs.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import { randomBytes } from 'crypto';
import { LogType } from './log_type.js';
import { UnencryptedL2Log } from './unencrypted_l2_log.js';

// ???????????????????????????????????
/**
* Data container of logs emitted in 1 function invocation (corresponds to 1 kernel iteration).
*/
Expand Down
3 changes: 1 addition & 2 deletions yarn-project/end-to-end/src/e2e_avm_simulator.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,10 @@ describe('e2e_nested_contract', () => {
it('Calls an avm contract', async () => {
const a = new Fr(1);
const b = new Fr(2);
// const expectedResult = a.add(b);

// TODO: fix type gen for avm_addArgsReturn - incorrect number of args
logger('Calling avm_addArgsReturn...');
await avmContact.methods.avm_addArgsReturn(a, b).send().wait();
logger('Success');
});
});
});
2 changes: 1 addition & 1 deletion yarn-project/simulator/src/public/executor.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@ import { Oracle, acvm, extractCallStack, extractReturnWitness } from '../acvm/in
import { AvmContext } from '../avm/avm_context.js';
import { AvmMachineState } from '../avm/avm_machine_state.js';
import { AvmSimulator } from '../avm/avm_simulator.js';
import { temporaryMapAvmReturnTypes, temporaryMapToExecutionEnvironment } from '../avm/temporary_executor_migration.js';
import { HostStorage } from '../avm/journal/host_storage.js';
import { AvmWorldStateJournal } from '../avm/journal/index.js';
import { temporaryMapAvmReturnTypes, temporaryMapToExecutionEnvironment } from '../avm/temporary_executor_migration.js';
import { ExecutionError, createSimulationError } from '../common/errors.js';
import { SideEffectCounter } from '../common/index.js';
import { PackedArgsCache } from '../common/packed_args_cache.js';
Expand Down
3 changes: 1 addition & 2 deletions yarn-project/types/src/abi/contract_artifact.ts
Original file line number Diff line number Diff line change
Expand Up @@ -130,8 +130,7 @@ function generateFunctionArtifact(fn: NoirCompiledContractFunction): FunctionArt
*/
function hasKernelFunctionInputs(params: ABIParameter[]): boolean {
const firstParam = params[0];
return (firstParam?.type.kind === 'struct' &&
firstParam.type.path.includes('ContextInputs'))
return firstParam?.type.kind === 'struct' && firstParam.type.path.includes('ContextInputs');
}

/** Validates contract artifact instance, throwing on error. */
Expand Down

0 comments on commit 4bf841b

Please sign in to comment.