Skip to content

Direct emissions chart and dashboard items#4730

Open
kndehaan wants to merge 14 commits into
emissionsfrom
emissions-user-input-output
Open

Direct emissions chart and dashboard items#4730
kndehaan wants to merge 14 commits into
emissionsfrom
emissions-user-input-output

Conversation

@kndehaan

@kndehaan kndehaan commented Jun 18, 2026

Copy link
Copy Markdown
Member

Context

This PR adds 1990 data to datasets, a chart and two dashboard items to the model as part of the direct emissions project.

Implemented changes

  • Direct total GHG emissions chart per sector
  • Dashboard items reduction in GHG emissions compared to 1990 and total future GHG emissions
  • Validation of 1990 queries (ETEngine) and future emissions queries (Mechanical Turk)

Related

Goes with pull requests:

Checklist

  • I have tested these changes
  • I have updated documentation as needed
  • I have tagged the relevant people for review

@kndehaan kndehaan changed the base branch from master to emissions June 18, 2026 11:50
@kndehaan kndehaan force-pushed the emissions-user-input-output branch from 020ac67 to 6d93070 Compare June 18, 2026 11:51
@kndehaan kndehaan changed the title Direct emissions GHG chart and dashboard items Direct emissions 1990 data, chart and dashboard items Jun 18, 2026
Base automatically changed from emissions to master June 19, 2026 09:44
@kndehaan kndehaan force-pushed the emissions-user-input-output branch from 2a34311 to 642e3da Compare June 19, 2026 10:04
@kndehaan kndehaan requested a review from mabijkerk June 19, 2026 10:20
@kndehaan kndehaan marked this pull request as ready for review June 29, 2026 10:17
@kndehaan kndehaan force-pushed the emissions-user-input-output branch from b4467e1 to 5bc737d Compare July 7, 2026 13:20
@kndehaan kndehaan changed the base branch from master to emissions July 7, 2026 13:21
@kndehaan kndehaan changed the title Direct emissions 1990 data, chart and dashboard items Direct emissions chart and dashboard items Jul 7, 2026
@mabijkerk

Copy link
Copy Markdown
Member

@kndehaan I think the new dashboard items should be selectable from the menu, not shown by default right? Now the bar is very cluttered:

image

@mabijkerk

Copy link
Copy Markdown
Member

Might it be better to put the direct GHG chart in the "Emissions" chart selection section while it is in beta, rather than in the "Overview" section?

image

@mabijkerk

mabijkerk commented Jul 9, 2026

Copy link
Copy Markdown
Member

Why are all the labels shown twice? Also, why is the red line shown twice in the 2050 emissions, why is it not shown for 1990?

image

@mabijkerk

mabijkerk commented Jul 9, 2026

Copy link
Copy Markdown
Member

Ideally you would also see a column for 1990 emissions in the table view of the chart.

image

@mabijkerk

Copy link
Copy Markdown
Member

I would add the new GHG chart as the default for the direct emissions download:
image

@kndehaan

kndehaan commented Jul 9, 2026

Copy link
Copy Markdown
Member Author

I think the new dashboard items should be selectable from the menu, not shown by default right? Now the bar is very cluttered

This is a caching issue (right @louispt1 ?). If you click on the dasboard items settings wheel and save again, it shows the "normal" 7 dashboard items. But it I will ask @louispt1 and @noracato again about this caching issue to ensure that this will not occur on beta or master.

Might it be better to put the direct GHG chart in the "Emissions" chart selection section while it is in beta, rather than in the "Overview" section?

Sure, I'll move it there.

Why are all the labels shown twice? Also, why is the red line shown twice in the 2050 emissions, why is it not shown for 1990?

This is a bug that was already solved in one of the commits. Are you sure your local branch is up-to-date with remote? I don't see the duplicate labels:

image

Ideally you would also see a column for 1990 emissions in the table view of the chart.

I agree. I'll ask the devs to see if this would be very easy to implement.

I would add the new GHG chart as the default for the direct emissions download

Good point! I'll do that.

@mabijkerk

Copy link
Copy Markdown
Member

If you click on the dasboard items settings wheel and save again

I cannot see the wheel, so this does not work for me.

@mabijkerk

Copy link
Copy Markdown
Member

This is a bug that was already solved in one of the commits. Are you sure your local branch is up-to-date with remote? I don't see the duplicate labels:

Yes I'm sure, maybe I'm on the wrong branches?

  • etmodel: emissions-user-input-output
  • etsource: emissions-user-input-output
  • etengine: emissions-1990

@kndehaan

kndehaan commented Jul 9, 2026

Copy link
Copy Markdown
Member Author

The wheel becomes visible when you zoom out the page a lot. You're using the correct branches. I think strange things are happening here that might be related to caching as well? I think we should discuss it with the devs.

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

Successfully merging this pull request may close these issues.

4 participants