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

feature: Add --resolution option to flexmeasures show chart #1070

Merged

Conversation

victorgarcia98
Copy link
Contributor

@victorgarcia98 victorgarcia98 commented May 23, 2024

vl-convert library, which is used to render the Vega-Lite chart specs uses a embedded node engine which has a very low memory limit. So far, this is not possible not change. The memory limit is hit for large periods of data but we can downsample the data to a coarser resolution to reduce the memory usage.

How to test?

Asset Chart

flexmeasures show chart --start 2023-01-01T00:00:00+01:00 --end 2023-01-10T00:00:00+01:00 --asset 2387  --resolution P1D

Sensor Chart

flexmeasures show chart --start 2023-01-01T00:00:00+01:00 --end 2023-01-10T00:00:00+01:00 --sensor 16860  --resolution P1D

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on code under GPL or other license that is incompatible with FlexMeasures

@victorgarcia98 victorgarcia98 requested a review from Flix6x May 23, 2024 21:28
@victorgarcia98 victorgarcia98 self-assigned this May 23, 2024
documentation/changelog.rst Outdated Show resolved Hide resolved
flexmeasures/cli/data_show.py Outdated Show resolved Hide resolved
@victorgarcia98 victorgarcia98 requested a review from Flix6x June 3, 2024 12:45
@victorgarcia98 victorgarcia98 merged commit 472feda into main Jun 5, 2024
6 checks passed
@victorgarcia98 victorgarcia98 deleted the feature/cli/export-chart-with-custom-resolutions branch June 5, 2024 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants