Skip to content

73 water vapor notebook#74

Open
rogerkuou wants to merge 6 commits into
mainfrom
73_water_vapor_notebook
Open

73 water vapor notebook#74
rogerkuou wants to merge 6 commits into
mainfrom
73_water_vapor_notebook

Conversation

@rogerkuou

@rogerkuou rogerkuou commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Fix #73

Added check for empty dataset
Init loss in predict as a tensor
In the example notebook down sample landsurface mask by the factor of 2

@rogerkuou

Copy link
Copy Markdown
Collaborator Author

Hi @SarahAlidoost , while working on #73, I got an error in the notebook. This notebooks is adapted from the SST hourly one.

It fails on average_loss.item(), which hints that average_loss is a dictionany. Do you have clue on this? I am running with the CPU mode instead of GPU.

@rogerkuou
rogerkuou marked this pull request as ready for review July 16, 2026 13:52
@rogerkuou

Copy link
Copy Markdown
Collaborator Author

Hi @SarahAlidoost , can you review this notwbook when you have time?

The error was caused by an empty dataset. I added a check to predict, and initiated loss as an zero Tensor.

Another thing maybe we need to pay attention, is the water wapor data is downsampled by the factor of 2 comparing to SST. This requires some attention to lsm data. In the notebook I used an average window to downsample the lsm, and set <0.5 to 0, and >=0.5 to 1. We can discuss if this makes sense.

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 a example notebook for water vapor

1 participant