I noticed that default value for datum in the coops_product_within_region() function in stormevent.py, is None which will return STND datum (see example plot below).
Users can set datum=NAVD or any other value from the following list depending to their use
datum_list=['STND', 'MHHW', 'MHW', 'MTL', 'MSL', 'MLW', 'MLLW', 'NAVD']
I would suggest changing the default to NAVD or MSL as these two are more commonly used than STND, and/or adding the list of possible options to line 446 .
@SorooshMani-NOAA, Let me know your thoughts. If you don't mind, I can make these changes.
I noticed that default value for
datumin the coops_product_within_region() function in stormevent.py, isNonewhich will returnSTNDdatum (see example plot below).Users can set
datum=NAVDor any other value from the following list depending to their usedatum_list=['STND', 'MHHW', 'MHW', 'MTL', 'MSL', 'MLW', 'MLLW', 'NAVD']I would suggest changing the default to
NAVDorMSLas these two are more commonly used thanSTND, and/or adding the list of possible options to line 446 .@SorooshMani-NOAA, Let me know your thoughts. If you don't mind, I can make these changes.