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

checking constants #525

Closed
trontrytel opened this issue May 19, 2021 · 8 comments
Closed

checking constants #525

trontrytel opened this issue May 19, 2021 · 8 comments

Comments

@trontrytel
Copy link
Contributor

trontrytel commented May 19, 2021

https://github.com/atmos-cloud-sim-uj/PySDM/blob/fb619fa842ea419f6f038c532f07432f310f7e7a/PySDM/physics/aqueous_chemistry/support.py#L109

I'm not sure because of the comment left above this line, but shouldn't the value be 7.5 * 1e7?

@trontrytel
Copy link
Contributor Author

Also, not sure if I understand correctly the table 4 in Kreidenweis. But in libcloudph++, I think, for both dissolution and dissociation of HNO3 I was taking 8700 as the exponent factor. In PySDM its 0 right now.

https://github.com/atmos-cloud-sim-uj/PySDM/blob/fb619fa842ea419f6f038c532f07432f310f7e7a/PySDM/physics/aqueous_chemistry/support.py#L32

https://github.com/atmos-cloud-sim-uj/PySDM/blob/fb619fa842ea419f6f038c532f07432f310f7e7a/PySDM/physics/aqueous_chemistry/support.py#L45

@slayoo
Copy link
Member

slayoo commented May 19, 2021

Aniu! I have just rerun the fig 3 J&P example, not a hint of difference in results :(

@slayoo
Copy link
Member

slayoo commented May 19, 2021

and Fig 1 (K et al. example) looks as follows:
image
at least, here there is some "response" to the changed constants

@slayoo
Copy link
Member

slayoo commented May 27, 2021

k3 addressed in #538
Currently the plot looks well!
image

@trontrytel
Copy link
Contributor Author

Sounds good. Now we only have to convince ourselves that the way we treat Henrys law is correct...

@slayoo
Copy link
Member

slayoo commented May 27, 2021

image
:)

@trontrytel
Copy link
Contributor Author

Very cool!

@slayoo
Copy link
Member

slayoo commented May 27, 2021

the other two constants are now also changes in #538
Closing!

@slayoo slayoo closed this as completed May 27, 2021
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

2 participants