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

setting PATH in env vars section #136

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jrcain-usgs
Copy link
Contributor

Licensing

Adds instructions on how to set your PATH when installing ISIS.

Fixes #128

This project is mostly composed of free and unencumbered software released into the public domain, and we are unlikely to accept contributions that are not also released into the public domain. Somewhere near the top of each file should have these words:

This work is free and unencumbered software released into the public domain. In jurisdictions that recognize copyright laws, the author or authors of this software dedicate any and all copyright interest in the software to the public domain.

  • I dedicate any and all copyright interest in this software to the public domain. I make this dedication for the benefit of the public at large and to the detriment of my heirs and successors. I intend this dedication to be an overt act of relinquishment in perpetuity of all present and future rights to this software under copyright law.

@Kelvinrr
Copy link
Collaborator

Kelvinrr commented Mar 12, 2025

I don't know if this is addressing the issue.

When installing ISIS, you do not have to set your PATH variable. I think what @michaelaye was getting at was when doing development installs or legacy (non conda) installs. Or when you install ISIS in one environment and want to access it when the env is not activated.


conda env config vars set ISISROOT=$CONDA_PREFIX ISISDATA=[your data path]
conda env config vars set ISISROOT=$CONDA_PREFIX ISISDATA=[your data path] PATH=$PATH:$CONDA_PREFIX/bin
Copy link
Collaborator

@Kelvinrr Kelvinrr Mar 12, 2025

Choose a reason for hiding this comment

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

e.g., here, this would not accomplish much since $CONDA_PREFIX/bin is already in your PATH on activation.

Idk if we want to encourage people to set their PATH to another conda env after install, if you need that I imagine stacking would be better?

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

Successfully merging this pull request may close these issues.

ISIS docs: PATH addition
2 participants