Skip to content
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

Fix major issues in classic mode for deflagration model #72

Merged
merged 11 commits into from
May 28, 2024

Conversation

lukeshingles
Copy link
Member

@lukeshingles lukeshingles commented May 27, 2024

Major slowdown and strange bolometric light curve found for r10_d2.6_Z deflagration model in classic mode by @fionntancallan. The exact cause is unknown, but some combination of: thread_local storage of continuum opacities gnu::const/pure attributes, re-use of continuum opacities for similar nu_cmf, or not updating cellcache on thick cells.

@lukeshingles lukeshingles changed the title Fix: revert to v2023.05 closesttransition() to fix major issues in classic mode Fix major issues in classic mode by reverting some v2023.10 changes May 27, 2024
@lukeshingles lukeshingles force-pushed the develop_fixclassic_closesttransition branch from 5a9b047 to c247f82 Compare May 27, 2024 11:55
@lukeshingles lukeshingles reopened this May 27, 2024
@lukeshingles lukeshingles changed the title Fix major issues in classic mode by reverting some v2023.10 changes Fix major issues in classic mode related to continuum opacities May 27, 2024
@lukeshingles lukeshingles force-pushed the develop_fixclassic_closesttransition branch 2 times, most recently from f36165a to c9fe8a1 Compare May 28, 2024 08:05
@lukeshingles lukeshingles force-pushed the develop_fixclassic_closesttransition branch from beb6043 to d4a2dde Compare May 28, 2024 09:40
@lukeshingles lukeshingles changed the title Fix major issues in classic mode related to continuum opacities Fix major issues in classic mode for deflagration model May 28, 2024
@lukeshingles lukeshingles marked this pull request as ready for review May 28, 2024 12:33
@lukeshingles lukeshingles enabled auto-merge (squash) May 28, 2024 12:33
@lukeshingles lukeshingles merged commit d3921c6 into develop May 28, 2024
49 checks passed
@lukeshingles lukeshingles deleted the develop_fixclassic_closesttransition branch May 31, 2024 07:33
lukeshingles added a commit that referenced this pull request Jun 2, 2024
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