-
Notifications
You must be signed in to change notification settings - Fork 27
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
Missing anthropogenic forcing definition in 245 #40
Comments
Hey, did you solve this in the end or is this still an issue? |
Hey! This is still an issue in your notebook. I solved it by adding a similar block to the total forcing without volcanoes and solar forcing: for scenario in scenarios: |
Cool, if you want to make a pull request with the right changes in the notebook I can run it and merge it in if it works |
Ok, I will try ! Not so sure how to do it, but tell me if it works! |
In Notebook 245, the anthropogenic part of the forcing is missing in the forcing[scenario] dictionaries.
Probably missing something like the total forcing:
for scenario in scenarios:
forcing[scenario]['total'] = (
forcing[scenario]['co2'] +
forcing[scenario]['ch4'] +
forcing[scenario]['n2o'] +
forcing[scenario]['other_wmghg'] +
forcing[scenario]['o3'] +
forcing[scenario]['h2o_stratospheric'] +
forcing[scenario]['contrails'] +
forcing[scenario]['aerosol-radiation_interactions'] +
forcing[scenario]['aerosol-cloud_interactions'] +
forcing[scenario]['bc_on_snow'] +
forcing[scenario]['land_use'] +
forcing[scenario]['solar'] +
forcing[scenario]['volcanic']
)
The text was updated successfully, but these errors were encountered: