Skip to content

First usable if incomplete CI#1

Merged
JoanneBogart merged 43 commits intomainfrom
u/jrbogart/mini_ci
Nov 11, 2025
Merged

First usable if incomplete CI#1
JoanneBogart merged 43 commits intomainfrom
u/jrbogart/mini_ci

Conversation

@JoanneBogart
Copy link
Copy Markdown
Collaborator

Exercises two "create" scripts (create_main.py and create_flux.py to a degree by

  • invoking each with --help
  • invoking each in order to create trilegal catalogs

@JoanneBogart
Copy link
Copy Markdown
Collaborator Author

Although there is more to do here, in particular in the CI, I would like to merge now so that there is a usable version of the code on the main branch.

For the most part the code has been extracted with little or no change from the skyCatalogs repo. Other than minor clean-up, differences are

  • addition of partial exercise of catalog creation in the CI
  • fix of a long-standing bug involving overwriting old flux files

@JoanneBogart
Copy link
Copy Markdown
Collaborator Author

@jchiang87 There is now a test for comparing a newly-created cosmodc2 galaxy catalog to a standard as well as a similar test for TRILEGAL. I don't expect to do any more along these lines. Please take a look when you have a chance.

Copy link
Copy Markdown

@jchiang87 jchiang87 left a comment

Choose a reason for hiding this comment

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

I left a couple comments on lsst stack versions to use in the install docs and the conda requirements file, otherwise looks ok to me.

Comment thread docs/installation.rst
Load and setup the science pipelines
++++++++++++++++++++++++++++++++++++

First you need to setup the science pipelines. This involves sourcing a setup file and then using the Rubin *eups* commands to set them up.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It would be good to update the version of the Rubin code to use to something more recent than w_2024_20. I don't know what the earliest viable version is, but something that uses GalSim v2.7.2 would at least make things consistent with the most recen imSim release. Also, these instructions still point to /cvmfs/sw.lsst.eu/linux-x86_64 builds, which have been deprecated.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think w_2025_28, specified in the imSim conda_requirements, will be ok.

Comment thread etc/conda_requirements.txt Outdated
@@ -0,0 +1,8 @@
# conda install --file etc/conda_requirements should install all required dependencies of imSim including a conda based version of the Rubin pipelines.

stackvana>=0.2023.32
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think the minimal version specified here should match what's recommended in the install instructions.

@JoanneBogart JoanneBogart merged commit b60a56c into main Nov 11, 2025
2 checks passed
@JoanneBogart JoanneBogart deleted the u/jrbogart/mini_ci branch November 11, 2025 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants