Skip to content

[newchem-cpp] Introduce C++ wrappers for the interpolate subroutines#383

Merged
brittonsmith merged 3 commits into
grackle-project:newchem-cppfrom
mabruzzo:gen2024transcribe/interpolate-wrappers
Sep 12, 2025
Merged

[newchem-cpp] Introduce C++ wrappers for the interpolate subroutines#383
brittonsmith merged 3 commits into
grackle-project:newchem-cppfrom
mabruzzo:gen2024transcribe/interpolate-wrappers

Conversation

@mabruzzo

@mabruzzo mabruzzo commented Aug 20, 2025

Copy link
Copy Markdown
Collaborator

This was originally proposed as brittonsmith#37


This PR must be reviewed after #381 is merged


This is a small PR that introduces C++ wrappers for the interpolate subroutines.

These wrappers define function interface that closely match what the function signatures will look like for the transcribed versions of the functions (see #384).

@mabruzzo mabruzzo added the refactor internal reorganization or code simplification with no behavior changes label Aug 20, 2025
@mabruzzo mabruzzo moved this to Awaiting Review in New Chemistry and C++ Transcription Aug 20, 2025
@mabruzzo mabruzzo changed the base branch from newchem-cpp to main September 11, 2025 18:12
@mabruzzo mabruzzo changed the base branch from main to newchem-cpp September 11, 2025 18:12
@brittonsmith brittonsmith changed the base branch from newchem-cpp to main September 12, 2025 10:35
@brittonsmith brittonsmith changed the base branch from main to newchem-cpp September 12, 2025 10:36
@brittonsmith brittonsmith merged commit b9bbaa9 into grackle-project:newchem-cpp Sep 12, 2025
5 checks passed
@github-project-automation github-project-automation Bot moved this from Awaiting Review to Done in New Chemistry and C++ Transcription Sep 12, 2025
@mabruzzo mabruzzo deleted the gen2024transcribe/interpolate-wrappers branch September 12, 2025 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor internal reorganization or code simplification with no behavior changes

Projects

Development

Successfully merging this pull request may close these issues.

2 participants