Skip to content

Commit

Permalink
Merge pull request #90 from spacetelescope/release/v0.2.19
Browse files Browse the repository at this point in the history
Release/v0.2.19
  • Loading branch information
bhayden53 authored May 30, 2022
2 parents 42b12f0 + 85c33ec commit 72b3715
Show file tree
Hide file tree
Showing 6 changed files with 21 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ jobs:
run: |
source "$HOME/miniconda3/etc/profile.d/conda.sh"
conda activate caldp_${{ matrix.HSTCAL }}
export CRDS_CONTEXT=hst_1002.pmap
export CRDS_CONTEXT=hst_1015.pmap
pytest caldp --cov=caldp --cov-fail-under 70 --capture=tee-sys
- name: compute test coverage
Expand Down
15 changes: 13 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,8 @@ RUN yum remove -y kernel-devel &&\
patch \
curl \
rsync \
time
time \
which

RUN mkdir -p /etc/ssl/certs && \
mkdir -p /etc/pki/ca-trust/extracted/pem
Expand Down Expand Up @@ -89,4 +90,14 @@ RUN mkdir -p /grp/crds/cache && chown -R developer.developer /grp/crds/cache

# ------------------------------------------------
USER developer
RUN cd caldp && pip install .[dev,test]
# for any base docker image created later than and including stsci/hst-pipeline:CALDP_20220420_CAL_final,
# the critical base environment is now buried in a conda environment named "linux"
# this creates various issues with the docker run command
# I played around for several hours with ways to bury the conda activation in a .bashrc or .bash_profile,
# but I couldn't get docker to use it when the image was invoked with the docker run command.
# in the end, the least-risky way to fix the issue seems to be to hardcode the conda activation into the path
# and bake it straight into the image.
# --bhayden, 5-24-22
ENV PATH=/opt/conda/envs/linux/bin:/opt/conda/condabin:/opt/conda/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
RUN cd caldp && \
pip install .[dev,test]
2 changes: 1 addition & 1 deletion caldp/tests/test_all.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
# Set default CRDS Context
CRDS_CONTEXT = os.environ.get("CRDS_CONTEXT")
if CRDS_CONTEXT == "":
os.environ["CRDS_CONTEXT"] = "hst_1002.pmap"
os.environ["CRDS_CONTEXT"] = "hst_1015.pmap"

# For applicable tests, the product files associated with each ipppssoot below
# must be present in the CWD after processing and be within 10% of the listed sizes.
Expand Down
4 changes: 2 additions & 2 deletions changelog.md
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
- default base docker image set to CALDP_20220406_CAL_final
- default crds update to hst_1002.pmap
- default base docker image set to CALDP_20220527_CAL_final
- default crds update to hst_1015.pmap
2 changes: 1 addition & 1 deletion scripts/caldp-image-config
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ export CALDP_IMAGE_TAG=latest
export CALDP_DOCKER_IMAGE=${CALDP_IMAGE_REPO}:${CALDP_IMAGE_TAG}

# Fundamental calibration s/w image CALDP image inherits from
export BASE_IMAGE_TAG=CALDP_20220406_CAL_final
export BASE_IMAGE_TAG=CALDP_20220527_CAL_final
export CAL_BASE_IMAGE=stsci/hst-pipeline:${BASE_IMAGE_TAG}
6 changes: 3 additions & 3 deletions scripts/caldp-install-fitscut
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@ cd install-fitscut.tmp/
git clone https://github.com/healpy/cfitsio.git
cd cfitsio && git checkout 8838182 && ./configure --prefix=${PREFIX} && make && make install

wget http://tdc-www.harvard.edu/software/wcstools/wcstools-3.9.5.tar.gz
tar -zxf wcstools-3.9.5.tar.gz
cd wcstools-3.9.5 && make
wget http://tdc-www.harvard.edu/software/wcstools/wcstools-3.9.7.tar.gz
tar -zxf wcstools-3.9.7.tar.gz
cd wcstools-3.9.7 && make
mkdir -p ${PREFIX}/include/libwcs
cp libwcs/*.h ${PREFIX}/include/libwcs
cp libwcs/*.a ${PREFIX}/lib
Expand Down

0 comments on commit 72b3715

Please sign in to comment.