search

wangsg2526 / ROMI

0 stars 0 forks source link

Potential error in omi_utils.py #1

Open linyao1999 opened 1 year ago

linyao1999 commented 1 year ago

Hi,

When you calculate olr_clim in a leap year (lines 130-132), you may want to fill OLR for Feb 29, which is OLR[59,:,:] instead of OLR[60,:,:].

Best, Lin

wangsg2526 commented 1 year ago

Yes, you are right! Always minor bugs to fix. Thank you for catching this!

On Sat, Jan 7, 2023 at 3:35 AM Lin Yao @.***> wrote:

Hi,

When you calculate olr_clim in a leap year (lines 130-132), you may want to fill OLR for Feb 29, which is OLR[59,:,:] instead of OLR[60,:,:].

Best, Lin

— Reply to this email directly, view it on GitHub https://github.com/wangsg2526/ROMI/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACSQELP6PWYXCI37XVZZMDTWRBXZJANCNFSM6AAAAAATTMQNUU . You are receiving this because you are subscribed to this thread.Message ID: @.***>