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

updated bibliography #29

Closed
wants to merge 1 commit into from
Closed

updated bibliography #29

wants to merge 1 commit into from

Conversation

gidden
Copy link
Member

@gidden gidden commented Aug 12, 2019

Adds Huppmann et al., updating the landing page and both downloadable bibliographies. Addresses #18

@gidden gidden changed the base branch from master to 2019-update August 12, 2019 08:58
Copy link
Member

@khaeru khaeru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gidden can you please reformat per #20 (comment)

@@ -7,6 +7,8 @@ These webpages document the IIASA Integrated Assessment Modeling (IAM) framework

When referring to MESSAGE-GLOBIOM 1.0 as described in this document, please use the following citations (Download :download:`ris </_static/message-globiom.ris>`, :download:`BibTeX </_static/message-globiom.bib>`):

* Huppmann, D., Gidden, M., Fricko, O., Kolp, P., Orthofer, C., Pimmer, M., Kushin, N., Vinca, A., Mastrucci, A., Riahi, K., et al. (2019). The MESSAGEix Integrated Assessment Model and the ix modeling platform (ixmp): An open framework for integrated and cross-cutting analysis of energy, climate, the environment, and sustainable development. Environmental Modelling & Software 112, 143–156, `https://doi.org/10.1016/j.envsoft.2018.11.012 <https://doi.org/10.1016/j.envsoft.2018.11.012>`_.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...143–156, DOI: `10.1016/j.envsoft.2018.11.012 <https://doi.org/10.1016/j.envsoft.2018.11.012>`_.

@behnam-zakeri
Copy link
Contributor

@khaeru and @gidden, there is another PR #21, where the references are updated and new ones added to the bibliography list, and the text in all sections is updated accordingly. Also, the description of of the model is updated there in the section "Overview". I was wondering if you could review and comment on the PR #21 instead?

@jkikstra
Copy link
Contributor

jkikstra commented Oct 8, 2019

To me, it seems like these changes can just be copied and pasted into the changes in PR #21 (@GamzeUnlu95) and then this PR can be closed (as the updated bibtex is already in the 2019-update, and the updated RIS does not seem to have improvements).

So to point out the one thing that needs to be added in PR #21:

  1. source/index.rst : add changes from this PR (with the right syntax for the DOI)

@GamzeUnlu95
Copy link
Contributor

The reference name for the paper "The MESSAGEix Integrated Assessment Model and the ix modeling platform (ixmp)...." is different in 2019-update branch (huppmann_2019_MESSAGEix), in this PR (HUPPMANN2019143) and PR #21 (huppmann_message_2019). The version in PR #21 can be used as the final one as the text is also updated according to that.

@jkikstra
Copy link
Contributor

jkikstra commented Oct 9, 2019

@khaeru or @gidden, could one of you check and close this PR?

@khaeru
Copy link
Member

khaeru commented Oct 10, 2019

@GamzeUnlu95 @jkikstra your assessment seems correct; thanks for the great teamwork. Closing without merging.

@khaeru khaeru closed this Oct 10, 2019
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.

5 participants