-
-
Notifications
You must be signed in to change notification settings - Fork 348
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
3.0 liquid pool #1639
base: 3.0
Are you sure you want to change the base?
3.0 liquid pool #1639
Conversation
Thanks for the pull request! Could you please modify this PR so it targets the |
I apologized for the pull request. It is my first time to code with git and github. I did not realized when I made this PR. I folked Cantera to my own repo and I just planed to test on my own repo because these codes is still in development. I developed it for counterflow flame with liquid pool boundary condition at beginning of this year based on Cantera 3.0.0. I am not pretty sure if it still work when I change PR to main branch. I may need some time to make it compatible with main branch. |
No need to apologize! I think this is a feature that would be worth integrating into Cantera, once you're further along in the development. Please feel free to leave this PR open, but convert it to a "draft" PR -- there should be a link on the right side of the page that does this. That way, people will be aware that you're working on this, but will know that it isn't ready for an in-depth review yet. |
Hi speth, I appreciate for your suggestions. I have converted it to draft. After I finished development, I will request pull to main branch. |
Changes proposed in this pull request
If applicable, fill in the issue number this pull request is fixing
Closes #
If applicable, provide an example illustrating new features this pull request is introducing
Checklist
scons build
&scons test
) and unit tests address code coverage