dowlinglab / HFC-IL-thermodynamic-model-selection

MIT License
1 stars 0 forks source link

Organize MBDoE Analyses #17

Closed bbefort closed 1 year ago

bbefort commented 1 year ago

To finish up the MBDoE Analysis, we need to do the following:

@jialuw96 JW:

@dipperwang5 KW:

@bbefort BB:

bbefort commented 1 year ago

Screen Shot 2022-10-24 at 10 44 42 AM

bbefort commented 1 year ago

First column: 6 data points of prior info: (2, 5, 12, 19, 23, 26) Second column: 7 data points of prior info: (2, 5, 12, 16, 19, 23, 26) Third column: 8 data points of prior info: (2, 3, 5, 12, 16, 19, 23, 26)

dipperwang5 commented 1 year ago
  • Experiment Number output

finished

adowling2 commented 1 year ago

Does this have the correct scaling? I think there was a mistake. See my prior comments.

dipperwang5 commented 1 year ago

Does this have the correct scaling? I think there was a mistake. See my prior comments.

This should be the result after FIM dividing 1e-10

output
adowling2 commented 1 year ago

This was the original figure. I think the scaling is wrong. My intuition is for log10(det(M)) to be O(1) to O(20). Look through our discussion thread for the history of my comments.

jialuw96 commented 1 year ago

@adowling2 @bbefort Here are the heatmaps with the first 6 experiments:

image image image image

With 7 experiments: the new added one is marked by a star

image image image image

With 8 experiments: newly added one marked by a triangle:
image image image image

For 8 experiments E and ME, there are very slight changes in heatmap that are hardly visible; I have checked that I didn't mix up figures, these are the heatmaps for 8 experiments.

For the pressure heatmap with MPa instead of Bar: image

dipperwang5 commented 1 year ago

@jialuw96 Hi Jialu, I am wondering what is the correct $log_{10}$ D-optimality when considering scaling for all experiment?

adowling2 commented 1 year ago

@jialuw96 Can you make the symbols 50% larger. Please make the star and triangle black.

adowling2 commented 1 year ago

@dipperwang5 and @jialuw96 you should discuss how the change the FIM when we change the units from Pa to bar for the measurements. I think a 10 minute conversation will clear this up.

dipperwang5 commented 1 year ago

@dipperwang5 and @jialuw96 you should discuss how the change the FIM when we change the units from Pa to bar for the measurements. I think a 10 minute conversation will clear this up.

output

Hi Alex, this is the figure after scaling. I have done sanity check with Jialu's running result to make sure this is correct.

adowling2 commented 1 year ago

This makes a lot more sense. The model with 6 parameters is not identifiable with fewer than 6 experiments.

Can you make a second version of this plot with y axis bounds from -5 to 5 or similar? I want to clearly see the trend with >6 experiments.

dipperwang5 commented 1 year ago
output

Here is it.

adowling2 commented 1 year ago

Let's adjust the bounds to -4, 2 to look better. Also please increase the font size for everything. I'd like to include this in my talk tomorrow.

dipperwang5 commented 1 year ago

Let's adjust the bounds to -4, 2 to look better. Also please increase the font size for everything. I'd like to include th

output

is in my talk tomorrow.

adowling2 commented 1 year ago

@bbefort @dipperwang5 @jialuw96 I think we now have a FIM unit consistency issues in the presentation.

This slide suggests that AIC #1 model is not identifiable and that log10(D-optimilty) = sum_i( log10(eigenvalue_i) ) = O(8).

Screen Shot 2022-10-25 at 6 08 35 PM

Yet this slide shows log10(D-optimality) = O(1) with the updated scaling.

Screen Shot 2022-10-25 at 6 08 44 PM

What do we need to do to reconcile this? @jialuw96 Do you have a new FIM decomposition that is consistent with the scaling from @dipperwang5?

jialuw96 commented 1 year ago

@adowling2 Heatmaps with bigger markers for experiments:

6 experiments: image image image image

7 experiments: image image image image

8 experiments: image image image image

adowling2 commented 1 year ago

Great. Thank you. Let's also add these files as .png and .pdf to GitHub in a well organized location. We'll need them for the paper.

adowling2 commented 1 year ago

One more request. Can you add a toggle in your code to include a small '.' to show the locations of all possible experiments in the dataset? Finally, please see my prior comment about drawing the polygon.

jialuw96 commented 1 year ago

Great. Thank you. Let's also add these files as .png and .pdf to GitHub in a well organized location. We'll need them for the paper.

I added these figures .png in folder paramest/Heatmap_figures.

One more request. Can you add a toggle in your code to include a small '.' to show the locations of all possible experiments in the dataset? Finally, please see my prior comment about drawing the polygon.

Do you mean except for these markers for prior experiments, we add comparatively smaller markers "." for other experiments in the dataset that are not prior experiments for the figure we are saying? I added this toggle.

Polygon

I plotted one sample graph:

image

I used "quadratic" to connect edge points with curves.

adowling2 commented 1 year ago

@jialuw96 I would like you to add a small '.' to the plot to show each of the experiment locations in the dataset. Here is a similar plot:

image

But this uses stars instead of '.'. This plot shows us the experiments that are available to choose from.

Regarding the polygon, please see my comment here: #16

After you plot the data, draw a polygon that covers the region with pressures greater than 15 bar. This should hide the "jagged" edges caused by the NaNs. To specify the polygon, you will need to compute the edges. This is easy for the top and right edges which are straight lines defined by the boundaries of the plot. For the curved portion of the polygon, you will need to use 2D interpolation to find xR32 coordinates as a function of T that results in 15 bar pressure. When you use the 2D interpolation function, the x, y, and z values may not be what you expect immediately.

A few questions/comments about your plot:

jialuw96 commented 1 year ago

@adowling2 Regarding Polygon questions:

image

jialuw96 commented 1 year ago

@adowling2 Here is the candidate experiments pressure:

image

adowling2 commented 1 year ago

@jialuw96 Thank you. I want this information plotted on the contour plot you made. For experiments that are in the FIM prior, please use the large black circles. For the other experiments, please use a small grey '.'

jialuw96 commented 1 year ago

@jialuw96 Thank you. I want this information plotted on the contour plot you made. For experiments that are in the FIM prior, please use the large black circles. For the other experiments, please use a small grey '.'

I remade one of the heatmaps like this:

image

jialuw96 commented 1 year ago

Delieverables from Jialu: