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.

Sign up for GitHub

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

Jump to bottom

Fix bug #85

Merged

johanneskoch94 merged 2 commits into pik-piam:main from johanneskoch94:main

Jun 17, 2024

Member

johanneskoch94 commented Jun 17, 2024

Fix bug where scenarios were returned in PPP instead of MER
Bump version

johanneskoch94 added 2 commits

June 17, 2024 14:52


          Fix bug where scenarios were returned in PPP instead of MER

35b9dd4


          Bump version

5a9a8fc

johanneskoch94 merged commit bb9451d into pik-piam:main

3 checks passed

orichters reviewed

View reviewed changes

R/calcGDP.R

@@ @@ -82,7 +82,7 @@ calcGDP <- function(scenario = c("SSPs", "SDPs", "SSP2EU"), @@
                 }
                 # Convert to US$MER if required
-                if (grepl("US$MER", unit)) {
+                if (grepl("US\\$MER", unit)) {

Contributor

orichters Jun 17, 2024

Note: there is also the grepl("US$MER", unit, fixed = TRUE) option, which I find sometimes easier compared to escaping special characters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet