Skip to content

Composite overshoot#1003

Open
matthiasfabry wants to merge 7 commits into
mainfrom
composite_overshoot
Open

Composite overshoot#1003
matthiasfabry wants to merge 7 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants