-
Notifications
You must be signed in to change notification settings - Fork 24
fix(photochem): exclude flux from VODE convergence; hard-code atol for DTypeFront; add docs #1956
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
Merged
Merged
Changes from 8 commits
Commits
Show all changes
61 commits
Select commit
Hold shift + click to select a range
59cad2e
setup toml for testing
chongchonghe 9376b3f
use ben's integrator params
chongchonghe 7a7b66f
chore: set physically appropriate ODE tolerances in DTypeFront.toml
chongchonghe bf5a9da
fix: correct atol_rad_num to prevent VODE from stepping N_gamma negative
chongchonghe a50228c
nx=8
chongchonghe 0779a80
feat: subtract radiation floor from N_gamma passed to VODE chemistry
chongchonghe 7228371
fix: use T=1K physical Erad_floor so dark cells don't ionize; revert …
chongchonghe 6a2019a
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 97ad771
fix: lower Erad_floor to T=0.1K, remove timestep cap
chongchonghe 7ad29fe
update plan
chongchonghe b890266
feat(photochem): add SetAtolFromPhysics to derive VODE atol from phys…
chongchonghe 2ef204e
fix(photochem): use fixed radiation_failure_tolerance=0.01, lower Era…
chongchonghe e07527b
docs: show derived atol values as comments in DTypeFront.toml
chongchonghe eb18f0b
docs: document integrator parameters in parameters.md
chongchonghe d3c8bfb
fix(photochem): abort when no integrator tolerances are specified
chongchonghe 3168f80
docs: clarify when typical_* parameters are required in parameters.md
chongchonghe 1e64722
docs: simplify atol/N_floor check to temperature-only formula
chongchonghe 86366df
docs: rewrite photoionization.md for clarity and consistency
chongchonghe 52a23a2
docs: replace "1 ppm" with "one millionth" for clarity
chongchonghe 2608401
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 3ae4d47
fix(docs): add photoionization.md to mdBook SUMMARY.md
chongchonghe 2faacb8
cleanup
chongchonghe 84f5ac9
revert nx
chongchonghe 7eee4c6
ci: scope AMD GPU pipeline to DTypeFront only
chongchonghe 6045d2c
ci: scope CUDA pipeline to DTypeFront only
chongchonghe 1325b15
doc
chongchonghe f7f0651
test: set DTypeFront to 8^3 grid for GPU CI validation
chongchonghe cfd6019
Revert "test: set DTypeFront to 8^3 grid for GPU CI validation"
chongchonghe 94a6133
perf(photochem): exclude radiation flux from VODE convergence checks
chongchonghe ac94d46
update submodule Microphysics
chongchonghe 20febea
ci: add temporary scope comments to DTypeFront-only pipelines
chongchonghe 9c64393
docs: qualify radiation_failure_tolerance default for low-density env…
chongchonghe 533fc4f
docs: fix radiation_failure_tolerance rule of thumb
chongchonghe 02a5dc0
fix(photochem): derive atol_rad_num from T_min, keep radiation_failur…
chongchonghe 64e97f1
docs: describe typical_minimal_radiation_T as cold gas temperature
chongchonghe e64686b
set to 20 ts
chongchonghe 379fe09
docs: explain why flux is excluded from VODE convergence checks
chongchonghe 3897c74
docs: complete photoionization documentation, fix flux explanation
chongchonghe 00550cc
docs: use LaTeX math throughout photoionization documentation
chongchonghe fa30d5c
vode
chongchonghe e7d34c8
Merge branch 'development' into chong/photo-opt2
chongchonghe 814304e
update Microphysics
chongchonghe 6ad318e
refactor(photochem): use hard-coded atol/rtol in DTypeFront instead o…
chongchonghe aa3d85a
refactor(photochem): update atol/rtol for OneZone and Stromgren tests
chongchonghe 5e31e14
ci: scope GPU pipelines to DTypeFront and StromgrenSphere tests
chongchonghe 547a8fa
refactor(photochem): add species_failure_tolerance aligned with atol_…
chongchonghe 45001c1
perf(DTypeFront): raise atol_rad_num to 5e-2 and radiation_failure_to…
chongchonghe 9efd639
remove some unused parameters from dock
chongchonghe 08147cf
docs(photoionization): add beta definition, Krumholz 2007 cooling, an…
chongchonghe 707a1be
feat(photochem): guard against simultaneous resampled cooling and pho…
chongchonghe 9cd4b9f
fix(photochem): gate Jacobian column-5 entries on the rn[0] clamp
chongchonghe b5ff7b1
docs(photoionization): remove SetAtolFromPhysics/derived-atol sections
chongchonghe 55d912e
add bib
chongchonghe d9b4ddb
Address Ben's review: radiation_failure_tolerance factor, citations, …
chongchonghe ebf8002
Merge branch 'development' into chong/photo-opt2
chongchonghe f98d2f4
fix(Stromgren): set atol_rad_num same order as atol_spec per Ben's re…
chongchonghe d7a2c07
kurmholz :
chongchonghe c9c4a81
docs(photoionization): clarify species/radiation failure tolerance roles
chongchonghe ad12f38
Merge branch 'development' into chong/photo-opt2
chongchonghe cef8805
rewrite
chongchonghe ca45302
revert CI
chongchonghe File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.