Skip to content

Added SROCC, SRCCL and AR5 WGII (parts A and B)#35

Open
pmarbaix wants to merge 22 commits into
openclimatedata:mainfrom
pmarbaix:main
Open

Added SROCC, SRCCL and AR5 WGII (parts A and B)#35
pmarbaix wants to merge 22 commits into
openclimatedata:mainfrom
pmarbaix:main

Conversation

@pmarbaix

Copy link
Copy Markdown

Added SROCC, SRCCL and AR5 WGII (full report, parts A global and B regional). Thanks for all the work already done!

@rgieseke

Copy link
Copy Markdown
Contributor

Thank you for contributing, will take a look later!

@rgieseke

Copy link
Copy Markdown
Contributor

I pushed some changes to the doc creation process which should make it easier to review the changes. There were some warnings about fields from the BibTeX processing.

@pmarbaix

pmarbaix commented Mar 23, 2024

Copy link
Copy Markdown
Author

Thanks for the improvements. Regarding the warning, do you mean things like (?):

Database file #1: ../srocc.bib
Warning--can't use both author and editor fields in IPCC_2019_SROCC

If so, I have the impression that these are already present in your repository, and I followed the same approach. For example, if I create your files with make -B(= Unconditionally make all targets), I get

Database file #1: ../ar6-wg-i.bib
Warning--can't use both author and editor fields in IPCC_2021_WGI

This is because the reference 'IPCC_2021_WGI' is of type @book, while this type does not allow the authors and editors field simultaneously. I do not know why it is so, and could not find a clear documentation about it. https://ctan.mc1.root.project-creative.net/info/bibtex/tamethebeast/ttb_en.pdf indicates that @book requires an author OR an editor, but does not say that both cannot be provided simultaneously. As far as I understand, the warning does not appear with @incollection.

I am not familiar with GitHub yet, so I apologise if I missed something, but this is all I understand so far.
If I understand what you mean / want and the problem itself, then we have two choices: either continuing to live with that warning (it is nothing severe and some software do use both authors and editors), or we have to change @book to something else for all reports. I hope that this helps.

@rgieseke

Copy link
Copy Markdown
Contributor

Yeah, the warnings could be something to fix in general or maybe it's not possible. I spotted some in the new entries but haven't looked into it in detail.

One thing I saw while skimming was

R. (ed.) van Diemen. Annex i: Glossary. In P.R. Shukla, J. Skea, E. Calvo, V. Masson-Delmotte, H.-O. Pörtner, D. C., P. Zhai, R. Slade, S. Connors, R. van, M. Ferrat, E. Haughey, S. Luz, S. Neogi, M. Pathak, J. Petzold, J. Portugal, P. Vyas, E. Huntley, K. Kissick, M. Belkacemi, and J. and Malley, editors, Climate Change and Land: an IPCC special report on climate change, desertification, land degradation, sustainable land management, food security, and greenhouse gas fluxes in terrestrial ecosystems, book section Annex I Glossary. Cambridge University Press, Cambridge, UK and New York, NY, USA, 2019. 

in SRCCL. I think this should have IPCC as the author as van Diemen as the coordinating editor.

@pmarbaix

Copy link
Copy Markdown
Author

in SRCCL. I think this should have IPCC as the author as van Diemen as the coordinating editor.

Right, I corrected this. It is sometimes hard to get the official bib reference for IPCC reports in BibTeX. I hope that it is now good enough for the pull, thanks again.
Philippe.

@rgieseke

Copy link
Copy Markdown
Contributor

Thank you @pmarbaix! With the change in the sed command (see review comment) I can't build it locally on Linux. Can you confirm you are on Mac? I can also look into a solution that might work for both platforms (maybe with an environment variable or a check in the Makefile).

@pmarbaix

Copy link
Copy Markdown
Author

I apologise for the change to the Makefile (sed). I did not want to push it on GitHub but apparently made a mistake. Indeed it is needed on Mac. Of course a solution that would check and work on both would be wonderful (if that is not too long as compared to the objective - getting refs for the IPCC reports). The simple solution would be to remove my changes on the Makefile, as these were not intentional.

@rgieseke

Copy link
Copy Markdown
Contributor

Thank you! This builds now for me again locally. I'll review this and will then merge.

@rgieseke

rgieseke commented Apr 5, 2024

Copy link
Copy Markdown
Contributor

Sorry for not having yet reviewed this.
I guess this being somewhat large has led me to not knowing how long it will take to not starting ... 🙂

I'll split it up:

  • Review SROCC
  • Review SRCCL
  • Review AR5 WGII (parts A and B)
  • Review TAR WGII SPM
  • Fix merge conflicts

@rgieseke

Copy link
Copy Markdown
Contributor

I haven't had much downtime or long train rides to properly review this, so if anyone likes to help with that ...

Removed the entry for AR6 WGII Chapter 7 Supplementary Material: it does not exist.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants