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
ENH: Template mesh generation #40
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
83e2281
to
a0e1c17
Compare
a0e1c17
to
ca501fc
Compare
@@ -0,0 +1,1042 @@ | |||
{ |
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.
We should avoid the use of filter
for a variable name because it conflicts with the built-in function filter
. We can use filt
or filter_
or preferably something more specific, e.g. paster
in this case.
Reply via ReviewNB
@@ -0,0 +1,1042 @@ | |||
{ |
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.
We should prefer the snake_case
form with the snake case filters, e.g. OutputOrigin
-> output_origin
.
Reply via ReviewNB
@@ -0,0 +1,1042 @@ | |||
{ |
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.
Note, the output ends here with a KeyboardInterrupt
. In the future, we will want to save the version in the repository with the full output.
Reply via ReviewNB
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.
@tbirdso awesome!!
A few style-related comment inline for future reference.
Add notebook to demonstrate iterative mesh template generation with HASI registrar classes.
Depends on #29.