update skyCatalogs version; remove redundant package installs#505
update skyCatalogs version; remove redundant package installs#505
Conversation
…ebuilt lsst docker images now include batoid and batoid-rubin)
d58f0e4 to
33eb508
Compare
cwwalter
left a comment
There was a problem hiding this comment.
To the extent I understand the GitHub CI this LGTM.
Two notes just in case:
I remember that @jmeyers314 added that gitpython entry to fix some really hard to understand problem. Unfortunately, I can't remember what it was now...
It looks like this takes out all batoid installs? Do they get pulled in another way?
cwwalter
left a comment
There was a problem hiding this comment.
Accidentally didn't approve.
I see them in the docker images made by Rubin:
I seem to recall it being related to a skyCatalogs problem....anyway, that also seems to be in the docker image: |
|
Sorry I see you noted the batoid dependency in the beginning of the conversation. I was just looking at the files. |
Noting that the prebuilt lsst docker images now include batoid and batoid-rubin.