-
Notifications
You must be signed in to change notification settings - Fork 1
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
DM-29904: Update instructions to run in README #19
base: main
Are you sure you want to change the base?
Conversation
$ setup -r . | ||
$ source $LSST_HOME/loadLSST.bash | ||
$ setup lsst_distrib | ||
$ setup -j -r . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This suggests that there's something missing from the table file. You should never need to setup something else before -r .
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just tried it locally and setup -t bNNNN -r .
works fine for me and I'm pretty sure that I've asked Jenkins to build just pipelines_check
before without building anything else. pipelines_check
depends only on things that are already in lsst_distrib. @yalsayyad what error are you seeing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[yusra@lsst-devl01 ~]$ source /project/yusra/LVV-T362/lsst_stack/loadLSST.bash
(lsst-scipipe-0.7.0) [yusra@lsst-devl01 ~]$ cd /project/yusra/LVV-T362/lsst_stack/pipelines_check
(lsst-scipipe-0.7.0) [yusra@lsst-devl01 pipelines_check]$ setup -r .
setup: in file /project/yusra/LVV-T362/lsst_stack/pipelines_check/ups/pipelines_check.table: Product pipe_tasks not found
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you specify a tag does it work? setup -t TAG -r .
? I have no idea how current
interacts with this but maybe if EUPS doesn't have a current
tag for pipelines_check
it can't work out what tag to use for the dependencies?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(lsst-scipipe-0.7.0) [yusra@lsst-devl01 pipelines_check]$ eups list pipe_tasks
23.0.0+f2fbba1123 v23_0_0
git
(lsst-scipipe-0.7.0) [yusra@lsst-devl01 pipelines_check]$ eups list lsst_distrib
23.0.0+e2b4167800 current v23_0_0
(lsst-scipipe-0.7.0) [yusra@lsst-devl01 pipelines_check]$ setup -t current -r . # no
setup: in file /project/yusra/LVV-T362/lsst_stack/pipelines_check/ups/pipelines_check.table: Product pipe_tasks not found
(lsst-scipipe-0.7.0) [yusra@lsst-devl01 pipelines_check]$ setup -t v23_0_0 -r . # yes
(lsst-scipipe-0.7.0) [yusra@lsst-devl01 pipelines_check]$
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's really strange that there are only a few packages (pipe_tasks
being one) that do not have a current
tag:
0a1
> afw
6a8
> base
7a10
> coadd_utils
11a15
> ctrl_execute
15a20
> daf_base
16a22
> daf_persistence
26a33,34
> ip_diffim
> ip_isr
33a42,45
> meas_algorithms
> meas_astrom
> meas_base
> meas_deblender
51a64
> obs_test
52a66,68
> pex_config
> pex_exceptions
> pipe_base
53a70
> pipe_tasks
59a77,78
> shapelet
> skymap
60a80
> utils
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Manually running pipelines check again for verifying 25.0.0, and we're in the same situation. instructions say setup -r .
but no current tags for everything:
(lsst-scipipe-5.0.1) [yusra@sdfrome001 pipelines_check]$ setup -r .
setup: in file /sdf/data/rubin/user/yusra/DM-32561/lsst_stack/pipelines_check/ups/pipelines_check.table: Product pipe_tasks not found
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you mean to not specify the -j
in your setup
call? If lsst_distrib is already set up then pipelines_check should set up fine. There is nothing in the pipelines_check table file that isn't in lsst_distrib already.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you mean to not specify the -j in your setup call?
Adding the -j
is the whole purpose of this pull request.
- `setup -r .` would only work if the user had previously cloned a specific tag as current, which is not part of the basic pipelines installation instructions.
setup -r .
would only work if the user had previouslycloned a specific tag as current, which is not part of the
basic pipelines installation instructions.