-
Notifications
You must be signed in to change notification settings - Fork 56
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
Clean up equivalent polynomials implementation and cutfem method #1242
Conversation
…structor (equal to 0.0). When I switched memBase to Profiling.memLast to get rid of the NameError, it seemed to run fine. I don't know what the significance of memBase is, but it wasn't being used anywhere outside of log statements from what I could tell.
* rits-true option for linear solver convergence (exit on r or its) * safeguard against degenerate quad cut case (cut hits two vertices) * commented out hack to pin pressure dof for pure neumann problem
memBase from NumericalSolution.py is Profiling.memLast inside the con…
Codecov Report
@@ Coverage Diff @@
## master #1242 +/- ##
=======================================
Coverage 52.81% 52.82%
=======================================
Files 531 531
Lines 109772 109802 +30
=======================================
+ Hits 57975 58001 +26
- Misses 51797 51801 +4
Continue to review full report at Codecov.
|
@@ -1440,11 +1440,13 @@ def apply(self,A,x,y): | |||
zero_array = numpy.zeros(len(self.known_dof_is.getIndices())) | |||
|
|||
tmp2.setValues(self.known_dof_is.getIndices(),zero_array) | |||
tmp2.assemblyBegin() |
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.
What does the assemblyBegin() do? Was this a typo that it wasn't there before?
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.
The changes seem fine to me. However, I'm not too familiar with some of the files you modified. But all the checks have passed so this is good on my end.
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.
Thanks @ejtovar
Mandatory Checklist
Please ensure that the following criteria are met:
As a general rule of thumb, try to follow PEP8 guidelines.
Description