-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add download button for saving contours in the profile uncertainty
tab as columnar data
#176
Comments
yikes! |
I'm not able to reproduce the problem - if I run it on a FitProblem with 2 models I get something like this:
Can you give some more information about your fitProblem? If possible, can you attach a zip file with the data + model file here? Also, could you paste the first few lines of your problematic CSV output? |
Ah, it's the problem I had given you earlier in the week - but it might be because it's a MixedExperiment. The last sample in the list is a blank region of the substrate, so there is no irho or rhoM, and it is shared across all of the models. CSV attached - the headers have "Stack", not M0, M1, etc...
…________________________________
From: Brian Benjamin Maranville ***@***.***>
Sent: Friday, April 12, 2024 1:41 PM
To: reflectometry/refl1d ***@***.***>
Cc: Balakrishnan, Purnima P. (Fed) ***@***.***>; Author ***@***.***>
Subject: Re: [reflectometry/refl1d] Add download button for saving contours in the `profile uncertainty` tab as columnar data (Issue #176)
I'm not able to reproduce the problem - if I run it on a FitProblem with 2 models I get something like this:
"M0 z","M0 rho (68 lower)","M0 rho (68 upper)","M0 rho (95 lower)","M0 rho (95 upper)","M0 irho (68 lower)","M0 irho (68 upper)","M0 irho (95 lower)","M0 irho (95 upper)","M0 rhonM (68 lower)","M0 rhonM (68 upper)","M0 rhonM (95 lower)","M0 rhonM (95 upper)","M0 thetaM (68 lower)","M0 thetaM (68 upper)","M0 thetaM (95 lower)","M0 thetaM (95 upper)","M1 z","M1 rho (68 lower)","M1 rho (68 upper)","M1 rho (95 lower)","M1 rho (95 upper)","M1 irho (68 lower)","M1 irho (68 upper)","M1 irho (95 lower)","M1 irho (95 upper)","M1 rhonM (68 lower)","M1 rhonM (68 upper)","M1 rhonM (95 lower)","M1 rhonM (95 upper)","M1 thetaM (68 lower)","M1 thetaM (68 upper)","M1 thetaM (95 lower)","M1 thetaM (95 upper)"
-483.736,4.64920,4.65284,4.64953,4.65174,0.600167,2.06121,0.751717,1.63305,0.0000187382,0.00221306,0.000297606,0.00128383,269.514,269.810,269.514,269.746,-483.736,4.64920,4.65284,4.64953,4.65174,0.600167,2.06121,0.751717,1.63305,0.0000546714,0.00203070,0.000235010,0.00117346,269.514,269.780,269.514,269.711
-475.750,4.64920,4.65284,4.64953,4.65174,0.600167,2.06121,0.751717,1.63305,0.0000282566,0.00221306,0.000297606,0.00128383,269.513,269.810,269.514,269.733,-475.750,4.64920,4.65284,4.64953,4.65174,0.600167,2.06121,0.751717,1.63305,0.0000546714,0.00203070,0.000235010,0.00117346,269.513,269.780,269.514,269.707
-467.763,4.64920,4.65284,4.64950,4.65174,0.600167,2.06121,0.751717,1.63305,0.0000282566,0.00221306,0.000297606,0.00128383,269.356,269.810,269.514,269.733,-467.763,4.64920,4.65284,4.64950,4.65174,0.600167,2.06121,0.751717,1.63305,0.0000546714,0.00226157,0.000235010,0.00131939,269.356,269.780,269.514,269.707
Can you give some more information about your fitProblem? If possible, can you attach a zip file with the data + model file here?
—
Reply to this email directly, view it on GitHub<#176 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADRHOJXTIDNQFTDPUKZPGV3Y5AMGHAVCNFSM6AAAAABGDAKVI6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJSGE4TSNBVHE>.
You are receiving this because you authored the thread.
|
It looks like it's happening because the models have the same name ("Stack" for both). We should probably not use a dict keyed on the model name for passing around the profile data, because non-unique model names happen all the time. |
…collision with previous name... fixes #176
Tested with original model names: output is as follows
|
Current "Download CSV" button downloads only the first outputted contour file (e.g. rho, but not irho or rhoM, for only the first profile)
The text was updated successfully, but these errors were encountered: