Skip to content
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

Refactor Render1D #4757

Merged
merged 3 commits into from Mar 4, 2023
Merged

Refactor Render1D #4757

merged 3 commits into from Mar 4, 2023

Conversation

nilsvu
Copy link
Member

@nilsvu nilsvu commented Feb 19, 2023

Proposed changes

I've been using this to debug the elliptic DG operator at large outer radii in 1D. I added a bunch of new features to the Render1D script so it can produce plots like this now, including collocation points, element boundaries, Lagrange interpolation, and individual basis polynomials:

poisson

Here's another example animating a Burgers evolution with subcell:

burgers.mp4

Upgrade instructions

Code review checklist

  • The code is documented and the documentation renders correctly. Run
    make doc to generate the documentation locally into BUILD_DIR/docs/html.
    Then open index.html.
  • The code follows the stylistic and code quality guidelines listed in the
    code review guide.
  • The PR lists upgrade instructions and is labeled bugfix or
    new feature if appropriate.

Further comments

@nilsvu nilsvu force-pushed the py_cli_vis branch 5 times, most recently from d899296 to d589b88 Compare February 23, 2023 23:53
Also bind equality operator and add a test.
- Plot Lagrange interpolation in each DG element
- Optionally plot collocation points, element boundaries,
  basis polynomials
- Plot multiple variables (e.g. field and TCI status)
- Set plot style with a stylesheet if needed.
Copy link
Member

@nilsdeppe nilsdeppe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice :D

@nilsdeppe nilsdeppe merged commit 5bdd1e6 into sxs-collaboration:develop Mar 4, 2023
@nilsvu nilsvu deleted the py_cli_vis branch March 4, 2023 06:07
@nilsvu nilsvu added the new feature Adds a new feature that's worth highlighting in release notes label Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature Adds a new feature that's worth highlighting in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants