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

Laminar Flame Speed Functionality Addition #82

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

cailinmoore
Copy link

  • Added functionality to use laminar flame speed for model reduction
  • Added flame simulation file and modified existing files to work with laminar flame option
  • Added functionality to test flame additions

@rwest
Copy link

rwest commented Jan 11, 2023

Is this pull request fully working? (apart from documentation...)
Does it read from the yaml input file, or currently need to be accessed via the Python API?

@kyleniemeyer
Copy link
Member

Is this pull request fully working? (apart from documentation...)
Does it read from the yaml input file, or currently need to be accessed via the Python API?

This did work when it was submitted, but I've been holding off on merging because I'm in the middle of some major updates to pyMARS, including making it compatible with Cantera 2.6, and also replacing the CI dependence on Travis.

@GillesFBourque
Copy link

@kyleniemeyer,
Do you have a planned release date for the major update of pyMARS?
Thanks
Gilles

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.

4 participants