Feature #3171 conda documentation#3172
Merged
georgemccabe merged 9 commits intodevelopfrom Dec 8, 2025
Merged
Conversation
…t actually required
23 tasks
Collaborator
|
@georgemccabe Thanks for your work on this! I am an ignorant conda user. I ran
What do I need to do next to be able to access/use the tools? |
Collaborator
Author
|
Hi @jprestop, it looks like you don't have a conda environment activated. Did the log output say which conda env that metplus was installed into? You will need to activate that conda environment to get access to the apps. Once activated, they should be in your path as you expected. |
Collaborator
|
Thanks @georgemccabe. It says the following (among other output): |
Collaborator
|
I just tried the following (again, I am obviously an ignorant conda user): |
Collaborator
|
Below is the entire output from the run: |
Collaborator
Author
|
@jprestop , the install log output says:
which corresponds to the base environment, shown from your env list:
So you should run |
Collaborator
Can you please modify the instructions in the documentation to show how users can do that along with instructions to tell ignorant users like myself to run |
…a environemnt (if desired) and activating an environment before installing METplus. Also add supported platform and version information
Collaborator
Author
|
@jprestop, I just updated the instructions. |
jprestop
previously approved these changes
Dec 8, 2025
Collaborator
jprestop
left a comment
jprestop
left a comment
There was a problem hiding this comment.
Thanks @georgemccabe. I followed the new instructions and everything worked great. One minor suggestion is to change instances of "my_metplus_env" to simply "metplus_env".
I also reviewed the new README badges and ordering, and they look good. Thank you for your work on this task and the conda build overall! I approve this request.
github-project-automation
bot
moved this from 🔎 In review
to 🏁 Done
in METplus-7.0 Development
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Once this is reviewed, a corresponding PR should be made into main_v6.2 so that the content will be updated in the
latestversion of the documentation.Pull Request Testing
Describe testing already performed for these changes:
Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:
Review documentation changes -- is the content in the correct location? Is the information correct/sufficient?
http://metplus.readthedocs.io/en/feature_3171_docs_conda/Users_Guide/getting_started.html#conda
https://metplus.readthedocs.io/en/feature_3171_docs_conda/Users_Guide/installation.html#python-package-requirements
Review new README badges and ordering
Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes]
Do these changes include sufficient testing updates? [Yes]
Will this PR result in changes to the test suite? [No]
If yes, describe the new output and/or changes to the existing output:
Do these changes introduce new SonarQube findings? [No]
If yes, please describe:
Please complete this pull request review by 12/10/2025.
Pull Request Checklist
See the METplus Workflow for details.
Select: Reviewer(s) and Development issue
Select: Milestone as the version that will include these changes
Select: Coordinated METplus-X.Y Support project for bugfix releases or METplus-Wrappers-X.Y.Z Development project for official releases