Skip to content

73 water vapor notebook - #74

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

73 water vapor notebook#74
rogerkuou wants to merge 7 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.

@SarahAlidoost SarahAlidoost left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@rogerkuou Thanks! 👍 I added one comment on the changed code. Levante is not available today and I dont have data locally. I will check the notebook later. About the spatial resolution of water vapor, I expected era5 data have the same spatial resolution; this is something to check in our next update meeting. I couldnot also check the era5 data as CDS is also on maintenance.

Comment thread climanet/predict.py Outdated
@SarahAlidoost

Copy link
Copy Markdown
Member

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.

Indeed there are a few things about water vapor data to check:

  • the spatial resolution of data on levante is 0.5. The water vapor data is available on cds with resolution 0.25
  • data is cut between (-80, 80) in latitude dimension
  • the latitudes are not increasing they are from (80, -80) while longitude are i.e. (-180, 180)

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

2 participants