Skip to content
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-46249: Add SasquatchDatastore to enable uploading analysis_tools metrics #244

Merged
merged 2 commits into from
Feb 19, 2025

Conversation

BrunoSanchez
Copy link
Member

@BrunoSanchez BrunoSanchez commented Feb 17, 2025

Update ci run script to take namespace and url for sasquatch upload."

Add optional ChainedDatastore to host SasquatchDatastore

Add new parameters to accomodate for Sasquatch metric upload

{Summary of changes. Prefix PR title with JIRA issue.}


  • Do unit tests pass (scons and/or stack-os-matrix)?
  • Did you run ap_verify.py on at least one of the standard datasets?
    For changes to metrics, the print_metricvalues script from lsst.verify will be useful.
  • Is the Sphinx documentation up-to-date?

Update ci run script to take namespace and url for sasquatch upload."

Add optional ChainedDatastore to host SasquatchDatastore

Add new parameters to accomodate for Sasquatch metric upload
@bsmartradio bsmartradio self-requested a review February 18, 2025 23:36
Copy link
Contributor

@bsmartradio bsmartradio left a comment

Choose a reason for hiding this comment

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

Everything looks fine to me, and I did try running scons on the branch locally as well and my local stack is happy.

@BrunoSanchez BrunoSanchez merged commit 763dcd8 into main Feb 19, 2025
2 checks passed
@BrunoSanchez BrunoSanchez deleted the tickets/DM-46249 branch February 19, 2025 19:54
@BrunoSanchez BrunoSanchez restored the tickets/DM-46249 branch February 19, 2025 20:05
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