-
Notifications
You must be signed in to change notification settings - Fork 26
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
issue with symmetric grids #451
Conversation
See the pull request for documentation.
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #451 +/- ##
=======================================
Coverage 93.84% 93.84%
=======================================
Files 72 72
Lines 15008 15019 +11
=======================================
+ Hits 14084 14095 +11
Misses 924 924
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
If some of the grid spacing tests are a chore to review, it might be easier to just check results directly. For convenience: import sys
import os
sys.path.insert(0, os.path.abspath("."))
# sys.path.append(os.path.abspath("../../"))
from desc.grid import *
from desc.plotting import plot_grid
import numpy as np
theta = np.linspace() or integer
lg = LinearGrid(rho=, theta=, zeta=, sym=)
print(lg.spacing[:, 1])
plot_grid(lg) |
Fixes issues with symmetric grids.
LinearGrid
, specifying nodes at coordinatessym=True
.sym=True
, nodes at coordinatesLinearGrid
can now handle non-uniformly spaced nodes if the user passes them whensym=True
.grid.weights
onConcentricGrid
withsym=True
.