search

NREL / nsrdb

This repository contains all of the methods for the NSRDB data processing pipeline.
https://nrel.github.io/nsrdb/
BSD 3-Clause "New" or "Revised" License
5 stars 0 forks source link

Sanity check nsrdb time series frequency #11

Closed grantbuster closed 2 years ago

grantbuster commented 3 years ago

Using a frequency of "10m" instead of "10min" results in the error ValueError: zero-size array to reduction operation maximum which has no identity due using a 10 month frequency for interpolation. Possible verify that "h", "min", or "t" is in the freq string?

grantbuster commented 2 years ago

Maybe a utility function that cleans the input arg in this class: https://github.com/NREL/nsrdb/blob/b2f01d3afc37b142766de4bcafb5f60b5ab2641b/nsrdb/nsrdb.py#L65

Maybe here too: https://github.com/NREL/nsrdb/blob/b2f01d3afc37b142766de4bcafb5f60b5ab2641b/nsrdb/data_model/data_model.py#L125

Maybe here too: https://github.com/NREL/nsrdb/blob/b2f01d3afc37b142766de4bcafb5f60b5ab2641b/nsrdb/data_model/clouds.py#L758

bnb32 commented 2 years ago

the suffixes that we want to allow are 'min', 'h', and 'D'? or 't'? and should we assert that freq contains one of the above or should we replace 'm' with 'min' and allow routine to proceed?