Skip to content

Composite overshoot#1003

Open
matthiasfabry wants to merge 9 commits into
mainfrom
composite_overshoot
Open

Composite overshoot#1003
matthiasfabry wants to merge 9 commits into
mainfrom
composite_overshoot

Conversation

@matthiasfabry
Copy link
Copy Markdown
Contributor

add functionality to model a step + exponential overshoot.

you can enable with:

      overshoot_scheme(1) = 'step+exponential'
      overshoot_zone_type(1) = 'burn_H'
      overshoot_zone_loc(1) = 'core'
      overshoot_bdy_loc(1) = 'top'
      overshoot_f(1) = 0.335   ! step part, slightly less than brott
      overshoot_f2(1) = 0.01  ! exponential part, replace outer part of brott with this
      overshoot_f0(1) = 0.01

@matthiasfabry matthiasfabry added the enhancement New feature or request label May 19, 2026
@Debraheem
Copy link
Copy Markdown
Member

I just cleaned up some of the white space diff so this touches less files, and tweaked the docs slightly. Now we just need testing https://testhub.mesastar.org/composite_overshoot/commits/27af2e7.

@Debraheem Debraheem force-pushed the composite_overshoot branch from 27af2e7 to a84c8d0 Compare May 23, 2026 01:58
@wmwolf
Copy link
Copy Markdown
Member

wmwolf commented May 23, 2026

Running on Hercules: https://testhub.mesastar.org/composite_overshoot/commits/a84c8d0. It'll take a while to get through things since it's running serially, though.

@wmwolf
Copy link
Copy Markdown
Member

wmwolf commented May 24, 2026

Nearly done. Looks like make_pre_ccsn_13bvn is stalling... currently at over 10k models and log_dt ~ –10 in inlist_cc.

@wmwolf
Copy link
Copy Markdown
Member

wmwolf commented May 24, 2026

Hmm.. the more I watch make_pre_ccsn_13bvn, the more concerned I am. It seems to have no smooth failure mode like a maximum model number. I'll let it keep going, but I think you can probably count it as a failure. The remaining missing tests are from binary and astero, and thus haven't even started yet, so I doubt there are issues with them.

@Debraheem
Copy link
Copy Markdown
Member

It definitely needs a max_model_number. I think you should kill the test, if it's going on indefinitely. I'm a little surprised it is failing thought, given this branch did not touch it.

@Debraheem
Copy link
Copy Markdown
Member

@wmwolf https://testhub.mesastar.org/main/commits/aa27a08/test_cases/star/make_pre_ccsn_13bvn, it should only do ~4k timesteps. I can't see the log since it doesn't update testhub until the run is complete, but maybe this is helpful.

@wmwolf
Copy link
Copy Markdown
Member

wmwolf commented May 24, 2026

It surprised me, too. I killed it and the remaining tests should roll in soon.

@Debraheem
Copy link
Copy Markdown
Member

I'm surprised the BVN test is even failing, since it is untouched by this pr except via the change to

if (s%alpha_mlt(k)*Hp > dr) Hp = dr/s%alpha_mlt(k)

My lack of sleep from last week has turned into an illness, but when i'm back up to speed, i'll investigate and try to fix this while i go through all the other branches from the hackathon.

@wmwolf
Copy link
Copy Markdown
Member

wmwolf commented May 24, 2026

Feel better soon, @Debraheem! Hercules has finished testing. Everything passed, except for make_pre_ccsn_13bvn, which stalled at tiny timesteps and had to be manually killed.

@warrickball
Copy link
Copy Markdown
Contributor

I've just started this on bluebear, which does full tests. make_pre_ccsn_13bvn hasn't been presenting any issues in the full tests, so it might just be that an intermediate model needs updating.

@warrickball
Copy link
Copy Markdown
Contributor

All clear on bluebear.

@Debraheem
Copy link
Copy Markdown
Member

Debraheem commented May 26, 2026 via email

@warrickball
Copy link
Copy Markdown
Contributor

warrickball commented May 26, 2026

Incidentally, does this not also allow one to refactor pure step/exponential overshoot into special cases of step+exponential? As in, is step overshoot the same as step+exp with the exponential coefficients set to zero? (I haven't yet had a chance to look closely at the code.)

@Debraheem
Copy link
Copy Markdown
Member

Incidentally, does this not also allow one to refactor pure step/exponential overshoot into special cases of step+exponential? As in, is step overshoot the same as step+exp with the exponential coefficients set to zero? (I haven't yet had a chance to look closely at the code.)

I think in principle, yes. It might be much simpler as well. Something like just turning on the exponential tail if using step and f2 is non-zero.

@Debraheem
Copy link
Copy Markdown
Member

@warrickball following your comments, I've refactored the implementation into the step overshoot routine, and significantly reduced the diff. Thanks for the feedback! I've also regenerated the mod files from the bvn case, so it should pass normal testing now.

@Debraheem Debraheem added the bug Something isn't working label May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants