-
Notifications
You must be signed in to change notification settings - Fork 213
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
Reformatting Operations #187
Reformatting Operations #187
Conversation
Separate Hadamard from Code Main File
Reformatting ops
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.
(There are no actual updates to this file; I ran the formatter on the folder which appears to have updated it.)
It got a bit messy with the test reformatting, but here are some major updates:
|
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.
This file primarily has the core of the new updates (creation of op_name_dict, specific imports, etc.)
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.
reviewed
Reformatted operations to be in separate files. Will be happy to workshop how much or little separation we want in the files—right now, it's slightly arbitrary but should be more or less consistent. Once this format is decided, I'll edit the functionals file with similar updates.
Other misc. updates from this PR: