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.

Sign up for GitHub

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

Jump to bottom

Better docstrings, better variable names #7

Merged

MartinuzziFrancesco merged 2 commits into main from fm/naming

Nov 10, 2024

Owner

MartinuzziFrancesco commented Nov 10, 2024

Follow naming for initializers from Flux#2522
Not using in as a variable name
Added links from cells to wrappers and viceversa
Added arguments to wrappers as well

MartinuzziFrancesco added 2 commits

November 10, 2024 14:28


          some work on variable naming, docstrings

d6864db


          some tests ad fixes

4bacaff

MartinuzziFrancesco merged commit 656dd4d into main

3 checks passed

MartinuzziFrancesco deleted the fm/naming branch

November 10, 2024 14:20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet