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-39477: Set some batch job clustering defaults in an importable yaml in ap_pipe #141

Merged
merged 1 commit into from Jun 9, 2023

Conversation

kherner
Copy link
Contributor

@kherner kherner commented Jun 5, 2023

No description provided.

Copy link
Contributor

@erinleighh erinleighh left a comment

Choose a reason for hiding this comment

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

Testing using the clustering is in progress.

clustering_ApTemplate.yaml (Saha Bulge) run id 945726.0 and restarted run id 946542.0.
clustering_HSC_ApTemplate.yaml (HSC COSMOS) run id 965108.0.
clustering_ApPipeWithFakes.yaml (HSC COSMOS) run id 947741.0.
clustering_ApPipe.yaml (DECam) run id 959751.0.

Upon randomized inspection of the failures, I'm 99% sure it's my fault because I'm running all the runs at once, so I'm going to approve the changes! :)

s3df:
profile:
condor:
+Walltime: 43200
# Memory allocated for each quantum, in MB (if unspecified, default is 2048)
requestMemory: 4096
includeConfigs:
- ${CTRL_BPS_DIR}/python/lsst/ctrl/bps/etc/bps_defaults.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

This line is no longer needed.

@@ -7,11 +7,17 @@ project: ApPipe
campaign: DM-xxxxx

submitPath: ${PWD}/bps/{outputRun}
Copy link
Contributor

Choose a reason for hiding this comment

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

This line is no longer needed.

@@ -7,12 +7,18 @@ project: ApPipeMultiTractFakes
campaign: DM-xxxxx

submitPath: ${PWD}/bps/{outputRun}
Copy link
Contributor

Choose a reason for hiding this comment

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

This line is no longer needed.

s3df:
profile:
condor:
+Walltime: 43200
# Memory allocated for each quantum, in MB (if unspecified, default is 2048)
requestMemory: 4096
includeConfigs:
- ${CTRL_BPS_DIR}/python/lsst/ctrl/bps/etc/bps_defaults.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

This line is no longer needed.

@@ -9,16 +9,21 @@ project: ApTemplate
campaign: DM-xxxxx

submitPath: ${PWD}/bps/{outputRun}
Copy link
Contributor

Choose a reason for hiding this comment

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

This line is no longer needed.

@kherner kherner merged commit 9edf7e7 into main Jun 9, 2023
1 check passed
@kherner kherner deleted the tickets/DM-39477 branch June 9, 2023 00:46
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.

None yet

2 participants