slacgismo / efinsight Goto Github PK

When querying from data from the NREL API (particularly for ComStock) using the app UI, sometimes the notebook crashes from an HTTPError (404: Not Found), e.g. when querying marine climate ComStock data (building america data) which does not exist. Ideally the notebook should not crash and the exception should be handled gracefully.

This can be fixed in a couple of ways:

• Adjust what options are shown in the drop-down menu to only show where data is available (probably not the best way to handle this, as data availability might change).
• Add better handling of the exception thrown, and instead of the marimo app breaking, redirect to an available data option with a clear error statement of what wasn't available.

This should also be added at the sg2t level https://github.com/slacgismo/sg2t/blob/main/sg2t/io/loadshapes/nrel/api.py where these calls are being made.

<cell-14>:14: FutureWarning: 'H' is deprecated and will be removed in a future version, please use 'h' instead. /Users/dchassin/github/slacgismo/efinsight/lib/python3.11/site-packages/sg2t/utils/timeseries.py:111: FutureWarning: 'H' is deprecated and will be removed in a future version, please use 'h' instead. df_resampled = df_aggregated.resample('H').mean()