forked from AMReX-Astro/Castro
-
Notifications
You must be signed in to change notification settings - Fork 0
Update Castro #1
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
melilly
wants to merge
140
commits into
melilly:main
Choose a base branch
from
AMReX-Astro:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
also remove the old version of the subch inputs
Temperature was 0, causing tempi to be infinity during the eos call. I'm able to run wdmerger to the step # 3.
they are now `constexpr`
cellsize is not set at this point. So manually set it here.
…d of amrex (#3048) This pr adds explicit evaluation of the thermal diffusion source term in castro, without needing to use amrex. In particular, this seems to work well for 2D spherical geometry.
…t test (#3047) this pr adds problem_bc_fill.H, which allows to use inflow, i.e. Dirichlet, boundary condition by using the analytic solution for different problems. Even though this is intended to use for spherical 2d input file, but any input file theoretically can use this as long as its constant conductivity.
we had a ** due to line wrapping
* dataset processing can run in parallel * added code to make interactive plots of each dataset that show the raw slice data, the averaged profile, and the positions for each metric * metric thresholds are calculated using both the local maximum and global maximum at the same time (under <field>[<fac>%] and <field>[<fac>%]_gmax, respectively) * the local maximum for each field is stored in the output file, in the max_<field> column * if a field value never goes below the threshold for a metric, its position will be stored as NaN * --[xyz]lim arguments now work, despite some yt bugs * allow ignoring plotfiles after a certain time with --tmax * default FRB resolution is set to match the finest level, and is automatically adjusted if custom limits are used
…ffusion test (#3054) Updates problem_bc_fill to do either face-centered or cell centered depending on where castro.diffuse_use_amrex_mlmg is used. Gets 2nd order accuracy (with L-inf error) with max_level=0 for 2d spherica To test theta dependence, the analytic solution of spherical-bessel function and legendre polynomial is used. Currently only castro evaluation works. Also added an 2d spherical input file for noncenter Gaussian, a convergence test for this still needs to be done in the future.
if we mistakenly target "main" and then switch to "development", then check-ifdefs, codespell, and docs never run. This changes the action to look like the others so these should hopefully run.
some of the inputs were redudant also renamed a few updated the README.md finally added a dt option to the profiles.py script
we fill this in Microphysics, but it was not being captured in job_info
* 1D inputs and new model * update README * update 1d lower boundary
r is the reduction parameter but we were also using it in the reduction loop for the radius (for 2D spherical)
also disable a new check
…3061) this shows the mass fractions behind the detonation
add papers
we can't to .ptp() anymore, but instead should use np.ptp()
some loop counters were also uninitialized. this also fixes a few compiler warnings.
also remove some old inputs files
This was only an issue if we were doing 4th order and we were initializing the data as cell-averages directly.
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v4...v5) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
this adds Ryan's RNAAS to the section on shock burning
this now allows Ne20 and also the C/O ratio can be something other than 1/2. Each core X is specified separately now.
this uses a more robust solve for NSE
flagged by coverityt
This reads the data from the location of peak enuc and uses pynucastro to visualize the network flow
Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v5...v6) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
this small function can take 10% of the runtime of a simulation
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 6. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v5...v6) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
this gives problems on some machines that don't have a python3 command
(Castro.H -> timestep.H -> Rotation.H -> Castro.H)
this uses a better buffer size, adds the "ash" field, and cleans up some sizing
this is for xrb_spherical
this speeds up copy_cons_to_burn by about 10x.
this uses Array4.cellData() to reducing some indexing and uses a constexpr_for
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.