Addressing unequal times error from pvlib.temperature.prilliman with no leap day

Is your feature request related to a problem? Please describe. When using pvlib.temperature.prilliman(), if the weather data does not include leap day (e.g., using 2020 PSM3 from pvlib.iotools.get_psm3() with the default leap_day=False), it returns:

NotImplementedError: algorithm does not yet support unequal times. consider resampling your data.

This could be a common scenario and could be confusing to the user.

Describe the solution you’d like There are several possible solutions:

1. Modify the error text to always say something like NotImplementedError: algorithm does not yet support unequal times. consider resampling your data. check to make sure your data includes leap day for leap year(s).
2. Check to see if leap day is missing and tailor the error message accordingly (e.g., add leap day is missing from your data. to the error message).
3. Check to see if leap day is missing, automatically fill it in somehow (e.g., repeat Feb 28), then remove it from the results before returning the output time series.
4. Same as 3, but also notify the user that this has happened. Depending on time zone of the weather data, there might be a discontinuity in the resulting smoothed temperature after removing leap day (I think?).

Describe alternatives you’ve considered Manually fill in leap day or repeat weather data query with leap day included (e.g., use leap_day=True with pvlib.iotools.get_psm3()).

[kanderso-nrel]

Checking for missing leap days and trying to correct for them seems out of scope for temperature.prilliman to me. I agree with @adriesse, better to leave that can of worms to the user. But I am certainly in favor of improving that error message to mention leap days and perhaps data gaps in general. Maybe something like this?

algorithm does not yet support unequal times. consider resampling your data and checking for jumps from missing periods, leap days, etc.

We might also want to change “unequal times” to something else while we’re at it. Note that this same error message is used in detect_clearsky, so we should make sure the message continues to be appropriate for both functions.

As an aside, I suspect leap_day=False is rooted in matching the PSM3 API’s decision to set it False by default. But +1 to a new issue to discuss changing it on the pvlib side.

[AdamRJensen]

@williamhobbs in regards to:

Separately, I also like the idea of changing pvlib.iotools.get_psm3() so that the default is leap_day=True. Should that suggestion be opened as a separate issue? I wonder if there are any compelling reasons to keep it as False, other than the fact that other tools (e.g., NREL SAM) can’t handle leap day.

Then I recommend that is made a separate issue.