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

Add documentation of MESSAGE-MACRO tuning #345

Merged
merged 7 commits into from Apr 29, 2020
Merged

Add documentation of MESSAGE-MACRO tuning #345

merged 7 commits into from Apr 29, 2020

Conversation

khaeru
Copy link
Member

@khaeru khaeru commented Apr 27, 2020

Edited from original text by @OFR-IIASA.

Also, similar to iiasa/ixmp#281, the docs tree is 'flattened' so that more pages are visible at the first level in the sidebar, grouped under explanatory headings/captions.

How to review

PR checklist

  • Tests added. No code changes
  • Documentation added.
  • Release notes updated.

@khaeru khaeru added the docs Documentation label Apr 27, 2020
@khaeru khaeru added this to the 3.0 milestone Apr 27, 2020
@khaeru khaeru self-assigned this Apr 27, 2020
@codecov
Copy link

codecov bot commented Apr 27, 2020

Codecov Report

Merging #345 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #345   +/-   ##
=======================================
  Coverage   88.80%   88.80%           
=======================================
  Files          29       29           
  Lines        2555     2555           
=======================================
  Hits         2269     2269           
  Misses        286      286           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 09d887c...001e920. Read the comment docs.

@khaeru khaeru force-pushed the mm-document branch 2 times, most recently from ca7ae76 to 447b730 Compare April 27, 2020 17:31
@khaeru
Copy link
Member Author

khaeru commented Apr 28, 2020

@OFR-IIASA I've requested review from you particular but FYI @behnam-zakeri @danielhuppmann @volker-krey per e-mail yesterday, the URL in the description above points to a new page with the text from the e-mail.

I've made some edits for order and context, so please approve if the description is still accurate and faithful to the original instructions.

Copy link
Contributor

@OFR-IIASA OFR-IIASA left a comment

Choose a reason for hiding this comment

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

Thanks for creating this PR Paul from the intial text supplied. I have no further comments; looks good to me.

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

Successfully merging this pull request may close these issues.

None yet

2 participants