Skip to content

[newchem-cpp] calc_temp1d_cloudy,cool1d_cloudy: fix z interpolation bug and light cleanup - #587

Open
mabruzzo wants to merge 15 commits into
grackle-project:newchem-cppfrom
mabruzzo:ncc/simplify-calc_temp1d_cloudy
Open

[newchem-cpp] calc_temp1d_cloudy,cool1d_cloudy: fix z interpolation bug and light cleanup#587
mabruzzo wants to merge 15 commits into
grackle-project:newchem-cppfrom
mabruzzo:ncc/simplify-calc_temp1d_cloudy

Conversation

@mabruzzo

@mabruzzo mabruzzo commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

This PR does 2 things:

  1. the first commit fixes a redshift interpolation bug that I had accidentally introduced when I previously factored find z_index out of calc_temp1d_cloudy and cool1d_cloudy
    • I had been intending to write some explicit unit tests to check the control flow so that this sort of case never happens again, but never quite got to it. (And I think it's extremely important for us to merge in these bugfixes -- before we forget about them)
  2. Subsequent commits lightly clean up calc_temp1d_cloudy. Importantly we clean up the function enough that we no longer require any heap allocations.
    • it also makes one or two similar tweaks to cool1d_cloudy and cool1d_cloudy_old_tables (but those are not the focus of this PR)

@mabruzzo mabruzzo changed the title [newchem-cpp] Cleanup calc_temp1d_cloudy [newchem-cpp] calc_temp1d_cloudy,cool1d_cloudy: fix z interpolation bug and light cleanup Aug 16, 2026
@brittonsmith

Copy link
Copy Markdown
Contributor

@mabruzzo, I see you left a note about doing a few others things after PR #384 was merged. I'll let you take care of those and this merger conflict and then review.

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.

2 participants