Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scheduling with heterogeneous sensor resolutions #8

Closed
nhoening opened this issue Jan 29, 2021 · 1 comment · Fixed by #207
Closed

Scheduling with heterogeneous sensor resolutions #8

nhoening opened this issue Jan 29, 2021 · 1 comment · Fixed by #207
Labels
bug Something isn't working Scheduling

Comments

@nhoening
Copy link
Contributor

NB: this ticket is work in progress. I don't agree with my own analysis yet, but I couldn't afford to linger on this. In general we don't support scheduling under heterogeneous sensor resolutions, and there may be several issues at play here, like:

  • Arrival times falling between interval boundaries of the power sensor.
  • Missing resampling of prices to the resolution of the power sensor.

What is the problem?

Scheduling an asset with a sensor resolution of 5 minutes against market prices with a resolution of 15 minutes crashes the scheduler.

Analysis

The length of the price data is less than the length of the power schedule.

Speculation

The time range for the price query doesn't return all the prices that are needed. For example: scheduling per 5 minutes from 13:50 to 16:05 right now obtains prices per 15 minutes from 14:00 to 16:00. After resampling, len(prices) == 24, but len(schedule) == 27.

Suggested fix for speculated problem

The query should be filtered using start and end differently. Right now we have:

query = query.filter((cls.datetime > start - asset_class.event_resolution))  # dt > 13:35
query = query.filter((cls.datetime < end))  # dt < 16:05

Actually this seems correct already.

@nhoening nhoening added bug Something isn't working Scheduling labels Jan 29, 2021
@Flix6x
Copy link
Contributor

Flix6x commented Oct 31, 2021

I'd venture that PR #207 resolved this issue.

@Flix6x Flix6x linked a pull request Oct 31, 2021 that will close this issue
@Flix6x Flix6x closed this as completed Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Scheduling
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants