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

Use CADET C-API #6

Closed
2 tasks
schmoelder opened this issue Mar 27, 2022 · 1 comment · Fixed by #202
Closed
2 tasks

Use CADET C-API #6

schmoelder opened this issue Mar 27, 2022 · 1 comment · Fixed by #202
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@schmoelder
Copy link
Contributor

schmoelder commented Mar 27, 2022

Previously, CADET could only save simulation results to a file. However, this requires a lot of overhead and the IO can be limiting when many simulations are run (e.g. in optimization studies but also when many cycles need to be simulated to reach cyclic stationarity).
With its new C-API (still WIP), CADET also allows reading results from memory.

For this purpose, the methods for the following steps need to be implemented in CADET-Python and CADET-Process:

  • initialize simulator
  • set parameters
  • run simulator
  • read results

There still seem to be some small issues in CADET-Core / Python that need to be fixed before properly testing the C-API in CADET-Process:

@schmoelder schmoelder added enhancement New feature or request good first issue Good for newcomers labels Mar 27, 2022
@schmoelder schmoelder added this to the v0.10.0 milestone Dec 5, 2024
@schmoelder
Copy link
Contributor Author

I think, this is generally ready but we should test it again.

@schmoelder schmoelder mentioned this issue Dec 6, 2024
@schmoelder schmoelder linked a pull request Dec 6, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant