-
Notifications
You must be signed in to change notification settings - Fork 0
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
Merge mistakes #42
Merge mistakes #42
Conversation
Brought back charge field Fixed code that was erroring in Model (indices for mtx were wrong)
Updated the AnyonModel enum from the old python version to the new rust version
github.com:Cornell-QCA/anyon-braiding-simulator
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.
Looks great, thanks @Haadi-Khan!
Minor thing: I removed the anyon
and model
commands from the commands displayed when user inputs help
, but you added them back. Even though this is handled when checking the init_complete
flag, I didn't see any reason to confuse the user by providing these as options, since they'll never be used after initialization. Lmk if there's a reason you added them back, otherwise I'll update in my next PR (leaving it for now since it's small).
No description provided.