Skip to content

Some velocity field examples [WIP] #14

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Some velocity field examples [WIP] #14

wants to merge 8 commits into from

Conversation

The9Cat
Copy link
Member

@The9Cat The9Cat commented Apr 3, 2025

Three examples:

  1. Using the halo-disk EXP example
  2. Using a pure halo with a quadrupole bar
  3. Unstable Zang disk

All of these could use some tuning.

@The9Cat
Copy link
Member Author

The9Cat commented Apr 3, 2025

In halo bar.ipynb, one needs to get to N=10,000,000 particles to get nice fields. This is, in part, because I need R_max=4 to suppress the NFW l=1 halo mode so there are too few particles in the bar region. But, when one gets to these high values, it looks very nice!

I would propose that we add the coefficient files for this example without running the simulation as part of the example.

@The9Cat
Copy link
Member Author

The9Cat commented May 10, 2025

I had to remove cell content from zang disk vel to reduce the size to meet github constraints, alas.

…mputed disk halo example with a large number of particles
@The9Cat
Copy link
Member Author

The9Cat commented May 12, 2025

Recent updates include

  • Minor improvements in documentation
  • Debug testing stuff removed
  • Original notebooks tested against latest PR, Minor updates to field coefficient bases and OutVel EXP#143
  • Added a new notebook that exercises both disk and halo velocity fields using a precomputed set of coefficients. Might want to ditch the halo fields; except for density, they are not all that revealing, although they do provide an example of doing it...

To Do

  • More testing
  • The new notebook could be moved to Tutorials/Coefficients from Tutorials/EXP since an exp run is not part of the workflow but it seems to below with the others
  • More examples?
  • Once we are happy with these updates/changes, we need to make the navigation footers consistent with the tutorial 'course' ordering

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant