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

Update docs with links and a new paper #888

Merged
merged 4 commits into from
Feb 3, 2025

Conversation

glatterf42
Copy link
Member

@glatterf42 glatterf42 commented Nov 13, 2024

Thanks to @shreeyashn20 in #883, I have been made aware of the paper by @natiweber about the Brazilian energy model with seasonal pumped hydropower storage. This PR adds a link to that to the docs.
It also links our new community health files from our contributing page.

@natiweber or @behnam-zakeri (since you are a co-author): we usually use a preview image for the entries on our usage page. Please let me know if you have an image in mind that we can use for that :)

How to review

  • Read the diff and note that the CI checks all pass.
  • Look at the pages in the docs that changed and confirm they look good:

PR checklist

  • Continuous integration checks all ✅
  • [ ] Add or expand tests; coverage checks both ✅ Just updating docs.
  • Add, expand, or update documentation.
  • [ ] Update release notes. Just updating docs.

@glatterf42 glatterf42 added the docs Documentation label Nov 13, 2024
@glatterf42 glatterf42 self-assigned this Nov 13, 2024
Copy link

codecov bot commented Nov 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.6%. Comparing base (4c14473) to head (6fd03a8).
Report is 5 commits behind head on main.

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #888   +/-   ##
=====================================
  Coverage   95.6%   95.6%           
=====================================
  Files         48      48           
  Lines       4394    4394           
=====================================
  Hits        4204    4204           
  Misses       190     190           

@glatterf42 glatterf42 force-pushed the update/docs/bib-community-health branch from fac4740 to 1653084 Compare November 13, 2024 13:44
@natiweber
Copy link

natiweber commented Nov 14, 2024 via email

@glatterf42 glatterf42 force-pushed the update/docs/bib-community-health branch from 1653084 to 0d65d2d Compare November 18, 2024 07:20
@glatterf42
Copy link
Member Author

Thanks for the suggestions, let's see how we like the one I used now :)

@khaeru khaeru added this to the 3.10 milestone Jan 21, 2025
@khaeru khaeru force-pushed the update/docs/bib-community-health branch from 0d65d2d to 2181059 Compare January 21, 2025 13:14
@khaeru khaeru force-pushed the update/docs/bib-community-health branch from 2181059 to 6fd03a8 Compare February 3, 2025 09:33
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.

I am approving this as we would like to include this in the showcase of MESSAGE applications for the 3.10 release, which is already a little delayed.

@behnam-zakeri @natiweber if you have any objections to the image that @glatterf42 has picked out here, please let us know by end-of-day. If you don't manage to look at it, also no problem; we can make another PR to swap out the image for a different one.

@natiweber
Copy link

natiweber commented Feb 3, 2025 via email

@khaeru khaeru merged commit 1cf691c into iiasa:main Feb 3, 2025
25 checks passed
@khaeru
Copy link
Member

khaeru commented Feb 3, 2025

Great, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants