New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Model refactor api expr to circ #1238
Model refactor api expr to circ #1238
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only like a half-review for now:
can you please tell me what files I don't really have to pay attention to? i.e. which files are exact copy-and-pastes? Specifically the huge files like GenerateInputTree
StringUtils, Formats, and DataStructures are the only things unchanged. Core algorithm files were changed to accommodate Result. |
src/site/pages/digital/src/utils/ExpressionParser/GenerateInputTree.ts
Outdated
Show resolved
Hide resolved
src/site/pages/digital/src/utils/ExpressionParser/GenerateInputTree.ts
Outdated
Show resolved
Hide resolved
src/site/pages/digital/src/utils/ExpressionParser/TreeToCircuit.ts
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,24 @@ | |||
import {Graph} from "math/Graph"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's move this file into app/core/utils
I think
src/app/tests/Extensions.ts
Outdated
// } | ||
|
||
// const pass = bfs([source], depth); | ||
toBeOk(received: unknown) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can just say received: Result
and then you don't have to type-check it, I should probably do that for the others but I was probably referencing some js
src/app/tests/Extensions.ts
Outdated
// pass, | ||
// }; | ||
// }, | ||
toIncludeError(received: unknown, message: string) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
@@ -0,0 +1,43 @@ | |||
// TODO[model_refactor_api](trevor): Get this working for expression to circuit frontend |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not get this working now ?
closes #1264
closes #1273
closes #1277