Update install.rst #478
Merged
Update install.rst #478
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Update latest Python version Add footnote for #473
Codecov Report
@@ Coverage Diff @@
## main #478 +/- ##
=====================================
Coverage 98.5% 98.5%
=====================================
Files 42 42
Lines 4489 4489
=====================================
Hits 4425 4425
Misses 64 64 |
khaeru
approved these changes
Apr 24, 2023
There was a problem hiding this comment.
Looks great!
Note that these are the ixmp install instructions. We usually direct new users to the message-ix install instructions, and then ixmp gets installed as a dependency through that process—without them needing to read these separately. These are retained for completeness.
So, we should make a parallel PR with the same changes over there—would you be able to do that, or should I?
|
I can do that, but I've got to take care of some other things first. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add a footnote explaining the most crucial observation in and linking to #473. Also update the 'latest Python version' mentioned there. Note that our pyproject.toml only includes Python versions up to 3.10, so I chose that as the 'latest version'.
Due to a recent failure of our nightly CI tests on macOS (see more details on Slack and in issue #479), we also use this PR to pin version 4.2.3 of R.
How to review
PR checklist