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

pyjac2foam.sh issues #36

Open
pkpavalavanni opened this issue Nov 17, 2023 · 2 comments
Open

pyjac2foam.sh issues #36

pkpavalavanni opened this issue Nov 17, 2023 · 2 comments

Comments

@pkpavalavanni
Copy link

Hi,
I am trying to convert a detailed CRECK mechanism (492 species and 17790 reaction mechanism - http://creckmodeling.chem.polimi.it/menu-kinetics/menu-kinetics-detailed-mechanisms/107-category-kinetic-mechanisms/403-mechanisms-1911-tot-ht-lt).
While running the pyjac2foam.sh, it reports a formatting error, however, while looking at the reaction mechanism, it is not the case.
I am attaching the error and outfile along with the jacobian file where it is being reported as an error.

DLBFoam.zip

@kahilah
Copy link

kahilah commented Nov 18, 2023

Hi,

It seems that pyJac has generated a faulty line of c-code. Opening jacob_18.c in any code editor shows that you're missing parenthesis on line 15033. Either this error is introduced by pyjac python module or the file is edited afterwards. If the former is true, please file an issue to https://github.com/SLACKHA/pyJac

As an immediate action, you can try to fix the parenthesis manually by placing closing "))" at the of the line accordingly: ...dBdT[61])))))) * rho_inv; Please, check that parenthesis make sense from kinetics perspective.

Furthermore, as you're working with rather large chemical mechanisms, I'd like to encourage you to get familiar with the original publications of pyjac https://niemeyer-research-group.github.io/pyJac-paper/ as it is reported that you may get some challenges with compilers when problem size grows considerably.

@mahgadalla
Copy link

I think this is a pyJac bug, recall to have came across similar problem myself with an exotic kinetic mechanism. Had to fix it manually by appending parenthesis as noted above!

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

No branches or pull requests

3 participants