oceanmodeling / ondemand-storm-workflow

Other
2 stars 1 forks source link

latitude-dependent Coriolis #28

Closed yichengt900 closed 11 months ago

yichengt900 commented 12 months ago

@SorooshMani-NOAA I noticed in the current workflow that ncor=0 (constant Coriolis parameter) is used for the schism model configuration. Although I think it is not a big deal for most storm events, we might consider switching to ncor=1 (the latitude-dependent Coriolis) to fully gain the benefits of using large-scale surge simulation. FYI @FariborzDaneshvar-NOAA

FariborzDaneshvar-NOAA commented 12 months ago

See here for the impact of ncor: https://github.com/noaa-ocs-modeling/SurgeTeamCoordination/issues/109#issuecomment-1715988737

SorooshMani-NOAA commented 11 months ago

I need to update the pyschism version and also add ncor=1 to the config of the workflow.

SorooshMani-NOAA commented 11 months ago

I believe this has been already addressed. @FariborzDaneshvar-NOAA please let me know if you have noticed any issues with it, thanks!

FariborzDaneshvar-NOAA commented 11 months ago

@SorooshMani-NOAA All good, thanks!