physicsnemo icon indicating copy to clipboard operation
physicsnemo copied to clipboard

GraphCast loss function different from original DeepMind version

Open cr-metdev opened this issue 1 year ago • 1 comments

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?

cr-metdev avatar Jan 17 '24 17:01 cr-metdev

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: https://github.com/NVIDIA/modulus/pull/554

mnabian avatar Aug 06 '24 15:08 mnabian

Closing as complete

mnabian avatar Oct 18 '24 00:10 mnabian