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

CO2 net exchange chart is only presented for 24h and 72h - not for 30d, 12mo, all - missing totalCo2Export and totalCo2Import #7596

Open
consideRatio opened this issue Dec 20, 2024 · 2 comments
Assignees

Comments

@consideRatio
Copy link
Contributor

Bug description / Feature request

Data (totalCo2Export and totalCo2Import) isn't available for a CO2 net exchange chart with 30d+ views, so it's only presented for the 24h and 72h views. This was surprising to me since totalExport and totalImport is around. Is not calculating the CO2 equivalent to totalExport and totalImport data not done due to a bug, or is it a feature not yet developed?

Analysis

I'm not sure why totalCo2Export and totalCo2Import isn't calculated because I don't yet understand where it gets done. I figure parsers needs to acquire the info which it does for hourly levels, and then the hourly data needs to be processed to daily / monthly / yearly aggregates and that isn't getting done.

@consideRatio
Copy link
Contributor Author

consideRatio commented Dec 20, 2024

@VIKTORVAV99 and @tonypls, is the code calculating totalCo2Export not public?

I figure there are parsers doing work to acquire hourly data of relevance, but that there then is machinery to aggregate data to daily / monthly / yearly entries as well, and it seems that this is done for totalExport but not for totalCo2Export but I'm failing to spot what code to inspect to figure out whats wrong.

@VIKTORVAV99
Copy link
Member

@VIKTORVAV99 and @tonypls, is the code calculating totalCo2Export not public?

I figure there are parsers doing work to acquire hourly data of relevance, but that there then is machinery to aggregate data to daily / monthly / yearly entries as well, and it seems that this is done for totalExport but not for totalCo2Export but I'm failing to spot what code to inspect to figure out whats wrong.

All the calculations and data structuring is happening on our backend which is private.

I can take a look at why it's not included but we have the hourly data so should be possible to add if nothing else.

@VIKTORVAV99 VIKTORVAV99 self-assigned this Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants