Skip to content

Conversation

@LydiaFrance
Copy link
Contributor

@LydiaFrance LydiaFrance commented Oct 29, 2025

Closes #123 #103

More complex than I first thought, but managed to create a file in callbacks which looks through the Hydra config files to provide information about training and evaluation within the plot title. Apologies that this is a larger PR than I intended.

image

Caveat...

I have tried my best here but there is probably a better method for scraping the correct info through Hydra. I am not the best at knowing what might be the most useful info and checking it grabbed the right thing. Given our time limits for the showcase might be good to check this at a later date.

@github-actions
Copy link

github-actions bot commented Oct 29, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  ice_station_zebra/callbacks
  metadata.py 62, 66, 71, 89, 94, 112-113, 145-151, 192, 196-200, 228-229, 275, 328, 349, 436-437, 504, 506
  plotting_callback.py 100-101, 103-105, 113-117, 164-181
  ice_station_zebra/types
  simple_datatypes.py
  ice_station_zebra/visualisations
  layout.py
  plotting_core.py
  plotting_maps.py 170-172, 188, 196-198, 265-266, 328-330, 338-339, 360-364, 786, 819, 826, 838
Project Total  

This report was generated by python-coverage-comment-action

@LydiaFrance
Copy link
Contributor Author

Moved metadata to a footer instead. Made the footer optional.

Video

sea-ice_concentration-video-maps_1_241ea6f8743c2e0bf5be

Horizontal Colourbars

image (3)

Vertical Colourbars

image (2)

Without Footer

image

@LydiaFrance LydiaFrance requested review from IFenton and aranas October 30, 2025 13:46
@IFenton
Copy link
Contributor

IFenton commented Oct 30, 2025

Reviewing this

Copy link
Contributor

@IFenton IFenton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Basically looks great.

Just a thought on the exact text. Is the Epoch always the number of epochs that the model is trained on? Or is it something else (e.g. the epoch the checkpoint was created on). Trying to work out whether it should be 'Epochs' as a total or 'Epoch' as a specific one.

@LydiaFrance LydiaFrance merged commit e0d845c into main Oct 31, 2025
3 checks passed
@LydiaFrance LydiaFrance deleted the 123-plot-titles branch October 31, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add more informative title to evaluation plots

3 participants