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

Bug: align scenario names between company bubble and techmix for WEO2023 #23

Closed
MonikaFu opened this issue Nov 25, 2024 · 1 comment · Fixed by #25
Closed

Bug: align scenario names between company bubble and techmix for WEO2023 #23

MonikaFu opened this issue Nov 25, 2024 · 1 comment · Fixed by #25
Labels
bug Something isn't working help wanted Extra attention is needed priority

Comments

@MonikaFu
Copy link
Contributor

MonikaFu commented Nov 25, 2024

In one plot the scenario name is NZE 2050 (techmix) and in the other it is NZE_2050 (bubble chart).

To reproduce you can go to https://rmi-pacta.github.io/pacta-dashboard-svelte/ -> Company-level Analysis tab -> Go! -> change scenario for WEO2023 to NZE 2050 and then NZE_2050

I'm not sure if it is a data artifact of a problem coming from how the data is being prepared.

@MonikaFu MonikaFu added bug Something isn't working help wanted Extra attention is needed priority labels Nov 25, 2024
@cjyetman
Copy link
Member

both prep_key_bars_portfolio() and prep_key_bars_company() have this line that substitutes " " for "_" in the scenario names, which they inherited from pacta.portfolio.report:::prep_key_bars_portfolio() and pacta.portfolio.report:::prep_key_bars_company()

mutate(scenario = sub("_", " ", .data$scenario))

mutate(scenario = sub("_", " ", .data$scenario))

https://github.com/RMI-PACTA/pacta.portfolio.report/blob/5f002dc1267fe9ac66b379642d5465ec636bd30f/R/prep_key_bars_company.R#L70

https://github.com/RMI-PACTA/pacta.portfolio.report/blob/5f002dc1267fe9ac66b379642d5465ec636bd30f/R/prep_key_bars_portfolio.R#L60

That doesn't seem desirable here, so I'll submit a PR that removes those lines from the functions in this repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants