-
Notifications
You must be signed in to change notification settings - Fork 10
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
Replace the "normalization" scheme with Larch #43
Comments
@x94carbone I added the following class to the operations module. Is there a good way for me to test that it works?
|
@CharlesC30 yep you can test it using the |
@x94carbone Thanks! I just tested the class and made some changes to get it working. I also had to make some changes to the notebook to get it working on my end.
so I changed it to
Since it looks like the class is working fine I will submit a PR soon, then we could decide if we want to keep any changes to the notebook. |
@CharlesC30 perfect, thank you! That sounds great. Open a PR when ready! |
At Eli's request we will be replacing our
postprocessing
operators used for normalization with the scheme that is used in Larch. For example,@zleung9 FYI. I'll be working on this a bit with one of Eli's students.
The text was updated successfully, but these errors were encountered: