search

spacetelescope / science-platform-images

Other
4 stars 7 forks source link

updated packages, removed extraneous notebooks #52

Closed cganley1 closed 4 months ago

cganley1 commented 4 months ago

I am not sure why the jwebbinar check is failing. I was able to build successfully locally.

jaytmiller commented 4 months ago

If you still have your local image it'd be good to set FREEZE_CHILL=1, re-source setup-env, and type image-freeze. Then assuming there are requirements in deployments/jwebbinar/env-frozen you should add and commit those then push it up as part of this PR. Other than that, the build failure is a temporary mystery, roman and tike installed exactly the same package in theory... So update the frozen requirements and approved, thanks for taking care of this!

jaytmiller commented 4 months ago

I made a comment earlier about saving frozen requirements, then made the mistake of closing the PR. It's a good idea to save frozen requirements if you can get them easily, otherwise don't sweat it.

cganley1 commented 4 months ago

I had some issues with docker and capturing the frozen requirements. Those are now resolved and included the changes. Thanks @jaytmiller