-
Notifications
You must be signed in to change notification settings - Fork 76
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
possible problem with ies_phi_factor_file #265
Comments
Im guessing that really small initial concentration phi is doing weird things. Can you post your files somewhere for me to see what is going on? |
Unfortunately, I am affraid I cannot easily share my files, for confidentiality reasons. |
Yeah i was playing around earlier and noticed that I can generate some extreme measured phi values with weight adjustment but not the actual phi (the weight adjustment uses actual phi). So I think that means the noise realizations might be a place for you check... |
All good. Ill run a few more checks on my end. How are you generating the noise? Using standard_deviation in obs data csv? |
Yes, I confirm I use the standard_deviation of the the obs_data.csv file to set my obs noise. |
So here is where we got so far: And now, I write again to you because, just a few minutes ago, i tried to disable ies_phi_factor_file and noticed that the problem in case.obs+noise.csv remains. So this may not be related to ies_phi_factor_file but rather to the way case.obs+noise.csv is written. Best regards, |
I confirm that removing the space before "standard_deviation" solved the issue with the case.obs+noise.csv file. Also, returning to the topic which initiated this issue, I will soon provide you with inputs and outputs I can share so that you can understand why small concentration values cause a problem when activating ies_phi_factor_file. Best regards, |
ah ok. I thought the column names in the external files were having leading and trailing whitespaces trimmed off - I'll check that... |
If it does not contradict the purpose of PEST++, it may also be useful to only accept as column names a list of specific strings which are to be defined in a whitelist. But this is up to your choice ! |
Nah we want to allow unlimited additional columns in those external files as a means to carry metadata/aux information around thru the pest world...but the rec file does report explicitly when it finds something in an external file that it's going to use. So you can check for detection and usage... |
Hello,
I have a test case with observations of both piezometry (they are around 10² m) and concentrations (they are around 1e-9).
Also, the standard deviation of observations is set to 0.05 for all piezometries and to 1e-11 for all concentrations.
I use ies_phi_factor_file to change the weights so that the objective function is 50% piezometry and 50% concentrations.
I observe, something strange : the difference between the measured and actual objectives function became enormous as a result of the weight adjustment.
Before weight adjustment, the measured and actual objective functions are about the same order of magnitude :
--- pre-weight-adjustment initial phi summary ---
phi type mean std min max
measured 6233.62 7786.17 1031.19 23281.1
actual 5467.6 7613.79 166.446 22194.4
note: 'measured' phi reported above includes
realizations of measurement noise,
'actual' phi does not.
After weight adjustment, there is a 15 orders of magnitude difference :
--- initial phi summary ---
phi type mean std min max
measured 3.68989e+21 1.33153e+21 800702 4.68715e+21
actual 1.09352e+06 444877 625700 1.84434e+06
note: 'measured' phi reported above includes
realizations of measurement noise,
'actual' phi does not.
Does this look normal to you ?
Thank you for your help.
Best regards,
Benjamin JEANNOT
The text was updated successfully, but these errors were encountered: