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.

Sign up for GitHub

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

Jump to bottom

Replaced references to astroconda with conda-forge #137

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Replaced references to astroconda with conda-forge #137

wants to merge 1 commit into from

Conversation

shahm0
Copy link

@shahm0 shahm0 commented Feb 9, 2022

Also modified installation instructions to reflect switch to conda-forge.

@shahm0
Replaced references to astroconda with conda-forge
be969d4 
Also modified installation instructions to reflect switch to conda-forge.
sean-lockwood
sean-lockwood reviewed Feb 9, 2022
View reviewed changes
doc/source/gettingstarted.rst
Now we're ready to install the STScI Software Stack, we'll accomplish this by setting up a fresh Conda environment.
Run the following command, you can change "stisenv" to be whatever name for the environment you wish.

.. code-block:: sh

$ conda create -n stisenv python=3.7 stsci
$ conda create -n stisenv --file [insert URL of conda environment YAML file here]
Copy link
Member

Choose a reason for hiding this comment

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

Is there a recommended file?

@sean-lockwood
Copy link
Member

stistools (and our ancillary packages) are not yet in conda-forge, so we shouldn't merge this until that is resolved.

@shahm0
Copy link
Author

shahm0 commented Feb 14, 2022

stistools has a python package which is what the conda environment yaml file pulls, so the fact that it is not yet in conda-forge should not be an issue, I believe. The yaml file is currently being worked on, and it will probably be made available elsewhere that an external user can download from.

@nden
Copy link
Collaborator

nden commented Feb 15, 2022

The simplest way to install stistools is actually to do pip install stistools.

@shahm0 shahm0 marked this pull request as draft February 15, 2022 14:52
@shahm0
Copy link
Author

shahm0 commented Feb 15, 2022

Changed to a draft PR since we don't want to publish these new instructions until a new build is available.

@sean-lockwood
Copy link
Member

The simplest way to install stistools is actually to do pip install stistools.

stistools requires that hstcal be installed -- it accesses the c binaries through a subprocess. My understanding is that hstcal is not available to install via pip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sean-lockwood sean-lockwood sean-lockwood left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shahm0 @sean-lockwood @nden