search

amazon-science / earth-forecasting-transformer

Official implementation of Earthformer
Apache License 2.0
350 stars 60 forks source link

fix: incorrect usage of `calc_seq_hits_misses_fas` #4

Closed gaozhihan closed 1 year ago

gaozhihan commented 1 year ago

Issue #3

Change the incorrect usage of calc_seq_hits_misses_fas from hits, misses, fas = self.calc_seq_hits_misses_fas(target, pred, threshold) to hits, misses, fas = self.calc_seq_hits_misses_fas(pred, target, threshold) in the metrics used in SEVIR: https://github.com/amazon-research/earth-forecasting-transformer/blob/9c18c5a0be9c1af686827a191217a6cccf57812b/src/earthformer/metrics/sevir.py#L144