GiteonCaulfied / COMP4560_stokes_ml_project

A repository that we are going to use to keep track of project evolution, notes, ideas, etc.
1 stars 0 forks source link

Outcomes meeting 12/10 - Poster #10

Closed amartinhuertas closed 11 months ago

amartinhuertas commented 11 months ago
amartinhuertas commented 11 months ago

@sghelichkhani FYI ... find attached the poster specification instructions poster-spec.pdf

GiteonCaulfied commented 11 months ago

Hi @amartinhuertas,

I've updated the poster according to the outcomes of today's meeting.

  • @rhyshawkins reviews the scripts to plot the geoid surface and errors. There seems to be something wrong there.

I've checked my code again and it turned out I made a silly mistake when plotting the geoid errors that would make the error image all black, which means there's nothing wrong with @rhyshawkins 's code. I've replaced those geoid visualization in the poster and the report with correct ones, sorry about leading to some unnecessary confusion.

The POD table has been removed and I've replaced the current LOSS table with a box plot.

  • Remove the temperature plots of the lower animation from the poster. Increase the size of the images.

The lower animation (the LSTM one) has been removed and the size of the animation sheet has been increased.

... Use new extra space to compare visually how do they look at different time steps, for different values of S (rows time steps, columns the value of S, just a suggestion).

For this I think it may be better if I use row to represent different values of S (7 rows, where 6 of them are different values for S and one of them is for ground truth) and only shows 4-5 time steps in total using columns (start, 25th, 50th, 75th, end). Also, I plan to use a random simulation (e.g. the 657th simulation in the dataset) to compare the result for different values of S.

rhyshawkins commented 11 months ago

@GiteonCaulfied I've updated the example plotting script to use pyshtools which has the correct spherical harmonics CS phase and normalization. You will need to pip install pyshtools to use it.

amartinhuertas commented 11 months ago

I've checked my code again and it turned out I made a silly mistake when plotting the geoid errors that would make the error image all black, which means there's nothing wrong with @rhyshawkins 's code. I've replaced those geoid visualization in the poster and the report with correct ones, sorry about leading to some unnecessary confusion.

Ok, that makes sense.

For this I think it may be better if I use row to represent different values of S (7 rows, where 6 of them are different values for S and one of them is for ground truth) and only shows 4-5 time steps in total using columns (start, 25th, 50th, 75th, end). Also, I plan to use a random simulation (e.g. the 657th simulation in the dataset) to compare the result for different values of S.

Yes, whatever you think it is best to illustrate visually the outcome of your research.

amartinhuertas commented 11 months ago

Some additional comments:

amartinhuertas commented 11 months ago

Figure 7 caption. Replace "Best Geoid visualization" by "Visualization of most accurate Geoid prediction"

GiteonCaulfied commented 11 months ago

Hi @amartinhuertas @rhyshawkins ,

The poster and report has been updated.

When I was visualizing the plots of animations with different values of S, I realized that I made a mistake in my implementation that leads to the POD table we were struggle to understand during this week meeting. The mistake has been fixed and I've uploaded some brand new table for the Data loss, POD difference and relative POD difference in the last section of the Mantle Convection Chapter in my report (including the box plots).

Here's the correct result for your convenience:

Screenshot 2023-10-13 at 5 29 54 pm

Fortunately, the result now makes more sense. The animation sheet is also added to the poster. You can also find it in the following path of the repo: 2D_GIFs/larger_dataset(interpolated)/FNN_S_consecutive.gif

  • In the geoid plots, you are not showing the color palette. Thus, we cannot know, e.g., the magnitude of the errors.

The color palettes are now shown for both the visualization image and the error image using @rhyshawkins 's latest code.

  • Not a big deal: can you make all figures with transparent background? png format supports transparent background.

I've made some of the figures transparent and I will head for others later.

UPDATE: All figures (except for the first one) have been made transparent.

  • why xAT in the definition of the affine map, and not just Ax. The second is easier to grasp, I think.

xAT is used here to be consistent with Pytorch's official document on Linear layer.

Here's the document: https://pytorch.org/docs/stable/generated/torch.nn.Linear.html

Figure 7 caption. Replace "Best Geoid visualization" by "Visualization of most accurate Geoid prediction"

The caption has been changed, thanks for the naming suggestion!

rhyshawkins commented 11 months ago

I've been trying to find a good general paper/reference on the geoid problem. A good short and high level introduction is the following Nature letter: Hager, B. The geoid and geodynamics. Nature 299, 104–105 (1982). https://doi.org/10.1038/299104a0

The key points are that:

There is a book reference, but early work on this problem can be found in the following reference. This is essentially what the geoid code is doing but with some extra bits that I may have to talk to Sia about.

https://doi.org/10.1029/JB089iB07p05987

GiteonCaulfied commented 11 months ago

Hi @rhyshawkins @amartinhuertas,

A poster with more Geoid related content has been pushed to the repo. I added a few paragraphs and one figure to the geoid part of Introduction, Research Aims, Conclusion and Future Works based on the key points @rhyshawkins mentioned. I haven't updated the geoid part of the report with these background yet and I will do this soon.

amartinhuertas commented 11 months ago

Some comments:

amartinhuertas commented 11 months ago

Add an "Acknowledgements" section with the following content:

This work was supported by computational resources provided by the Australian Government through the National Computational Infrastructure (NCI) under the ANU Merit Allocation Scheme.

amartinhuertas commented 11 months ago

Latent space size in the figure does not correspond to the one specified in the text. Please fix.

GiteonCaulfied commented 11 months ago

Hi @amartinhuertas,

I've updated the poster and pushed it to the repo.

  • Uniform usage of periods to end sentences. You use it in some sentences, but not in all sentences. I am not sure if in posters one uses periods to finish sentences in bullet points (to check)

All sentences have periods now for consistence

  • Fig8: What geoid plots represent? (magnitude and units)

Some description about magnitude and units has been added under the sample geoid visualization figure for clearance according to the key points listed by @rhyshawkins

  • Fig8: All 4 geoid plots seem the same
  • Fig9: Why error on top in black? Are you using the same scale for both plots? I would use different scales for each plot, and thus different color bars.

I've now used separate scales for all six plots, even though 4 plots seem the same, there are indeed some difference between their scales. The error on top is all black because the error is too small and I am using the same scale for both error plots, this is fixed now.

  • Fig10: It is not clear (to me) what you are comparing there and how the figure should be read. For example, given a figure, how can I now the corresponding time-step?
  • Fig12: the same as Fig 10. I would explicitly put in the figure the values of S and T in the bottom and left margins. It should be easy to identify the value of S and the associated time stamp for each plot in the matrix.

I've added some description under these figures to make things a bit more clear, will try to update the figures themselves such as putting in the figure the values of S and T in the bottom and left margins later.

  • Fig 11: I would remove S=99, put it in another figure, or leave all the values together in the same plot, but zoom the initial values of S. Right now, there are two scales in the figure, and the reader cannot distinguish the differences among the first five values of S tested.

The S=99 is now removed and put in a separate figure. As expected, the first five values of S look better now.

Add an "Acknowledgements" section with the following content:

This work was supported by computational resources provided by the Australian Government through the National Computational Infrastructure (NCI) under the ANU Merit Allocation Scheme.

The "Acknowledgements" section has been added, thanks for the suggestion!

Latent space size in the figure does not correspond to the one specified in the text. Please fix.

Sorry about that, this is fixed now.

amartinhuertas commented 11 months ago
GiteonCaulfied commented 11 months ago

Hi @amartinhuertas,

I've pushed the updated report with some geoid background to the repo. Also, Fig12 and Fig10 in the poster now are added with some lines and text to make things more clear.

  • Typo in Fig. 11 title
  • Period in the caption of figures.

This is fixed now, period are also added to the caption of figures in the report.

amartinhuertas commented 11 months ago

I think we are still missing in the poster a non-expert motivation/big picture of the project.

Some ideas (to be refined):