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

collision partner IDs #192

Closed
migueldvb opened this issue Nov 15, 2016 · 8 comments · Fixed by #215
Closed

collision partner IDs #192

migueldvb opened this issue Nov 15, 2016 · 8 comments · Fixed by #215

Comments

@migueldvb
Copy link
Contributor

The user parameter collPartIds is used to match the densities of collisional partners with the rates listed in the LAMDA data file. I have collisional partners that are not one of the 7 species defined in LAMDA ("H2","p-H2","o-H2","electrons","H","He","H+") so I am reusing those IDs for matching the datafiles to the density values. It looks like this should work for any molecules when collPartIds and nMolWeights are set but I may have missed something. Is there any reason why using different collisional partners from the valid LAMDA ones may break something?

@tlunttil
Copy link
Contributor

As far as I know, it should work; for example, there's no special 'H2-ness' in collision partner ID 1. But @allegroLeiden knows these parts of code much better than I do.

@tlunttil
Copy link
Contributor

Some of the messages do refer to the default LAMDA partner names, but that doesn't affect calculations.

@migueldvb
Copy link
Contributor Author

That is what I thought, thanks!

@allegroLeiden
Copy link
Contributor

Sorry not to comment before but I have been working pretty hard trying to recover from a multiple disk failure.

Technically I think this ought to work but I don't think it is a very elegant way to do things. I'll have a think about it when I have some time and see if I can't come up with a better way. We definitely ought to have a mechanism to accommodate users who don't want to be restricted to LAMDA.

@migueldvb
Copy link
Contributor Author

Using collPartIds greater than 7 does not work at the moment because readMolData tries to match the collision partner with one of the LAMDA values.

@tlunttil
Copy link
Contributor

As a quick fix, I think it would be enough if you change the value of MAX_N_COLL_PART in lime.h and add names for new collision partners to *collpartnames[] in molinit.c. But it's true that the hardcoded LAMDA stuff should be removed in future.

@migueldvb
Copy link
Contributor Author

@tlunttil that is a good workaround.

@allegroLeiden
Copy link
Contributor

Maybe we could allow the user to submit a list of names and default to LAMDA if it is not supplied?

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 a pull request may close this issue.

3 participants