-
Notifications
You must be signed in to change notification settings - Fork 52
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
Accessing and modifying model parameters via REPL - SAFTgammaMie #145
Comments
Hi, if you are updating directly the values on the REPL, the ones actually used in the calculation are the ones in
If you don't modify the number of groups or sites, and in this particular case that is correct. we have something of a general function, By the way, you can modify directly the pair parameters and symmetry is respected:
It shouldn't, but just to be sure, can you send what are your model values? ( |
The dispersion data is fine, because the values in that VR model are GC-averaged (we move from 4 groups to 2 components) . In the other part, there is definitely something going on with the association. Can you send me the outputs of |
ok, i can reproduce your bug. reproducer:
|
Great, thanks for the time! Do you think this affects calculations / is hard to fix? PS: I was using recombine! and I noticed that interaction parameters calculated by default using combining rules (i.e. no entry in the databanks) are reset even if their value is set via model.params. This does not happen for the ones that have an entry. Is this behaviour intended? |
I plan to get a new release today with the fix.
Yes, we have a boolean flag for values that are not set in |
OK fantastic! I'll try the new version ASAP and play with the boolean flag. Thanks a lot for the support! |
This was fixed (with tests),so I'm gonna close this issue for now. |
Hi, I have been accessing and modifying the parameters of a water/PEG model (model = SAFTgammaMie(["water","PEG_1sit"])) in Clapeyron directly via the Julia REPL by using the "model.vrmodel.params" and "model.params" attributes. However, I have noticed that when there are many association sites the "model.vrmodel.params.epsilon_assoc.values" and "model.params.epsilon_assoc.values" vectors (same for bondvol) contain a different number of entries. In particular, it seems that the vrmodel one is missing some association values. Does this impact the calculations at all?
On another note, I have been trying to modify the parameters and was wondering if changing both the entries in "model.vrmodel.params" and "model.params" (accounting for symmetry) is sufficient to temporarily update a given parameter.
Thanks in advance for the help!
The text was updated successfully, but these errors were encountered: