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

dustfrac and dust-to-gas not correct when ndusttypes > 7 #61

Open
markahutch opened this issue Apr 20, 2022 · 0 comments
Open

dustfrac and dust-to-gas not correct when ndusttypes > 7 #61

markahutch opened this issue Apr 20, 2022 · 0 comments

Comments

@markahutch
Copy link
Contributor

When calculating the additional quantities "dustfrac" or "dust-to-gas" with more than 7 dust phases, the string length (80) is not long enough to parse the function. Unfortunately this does not throw an error, which makes it appear as though there is an error in the data file when there is not.

A quick fix would be to use a longer string length, but perhaps a better solution would be to add the ability to parse the string "sum(dustfrac)" as "dustfrac1+dustfrac2+..."

This would also help with working with deltav, e.g. when calculating differential velocities between dust phases.

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

1 participant