Don't resample timestamps if not needed.

salesforce / Merlion

Merlion: A Machine Learning Framework for Time Series Intelligence

BSD 3-Clause "New" or "Revised" License

3.43k stars 302 forks source link

Don't resample timestamps if not needed. #109

Closed aadyotb closed 2 years ago

aadyotb commented 2 years ago

Merlion forecasters have a property requires_even_sampling, which determines whether they should resample input timestamps. The changes in 1.2.0 made it so the timestamps would always be resampled, even if requires_even_sampling=False. This PR ensures this option is actually respected.