Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GraphCast loss function different from original DeepMind version #301

Closed
cr-metdev opened this issue Jan 17, 2024 · 2 comments
Closed

GraphCast loss function different from original DeepMind version #301

cr-metdev opened this issue Jan 17, 2024 · 2 comments
Assignees
Labels
question Further information is requested

Comments

@cr-metdev
Copy link

I noticed that the graphcast loss function in this repository differs from what was used in the original graphcast paper, since the implemented here does not weight loss based on pressure level or type of variable. Just wondering if there was a reason for this or if there are plans to implement the level/variable weighting in the graphcast loss function in future updates?

@NickGeneva NickGeneva added the question Further information is requested label Jan 18, 2024
@mnabian
Copy link
Collaborator

mnabian commented Aug 6, 2024

Hi @cr-metdev , thanks for posting your question. We started with a simplified version of the loss function, but now in a recent PR, the implementation has improved to include other loss weightings based on pressure level or type of variable: #554

@mnabian
Copy link
Collaborator

mnabian commented Oct 18, 2024

Closing as complete

@mnabian mnabian closed this as completed Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants