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

v1.4 canfar run #690

Open
wants to merge 403 commits into
base: develop
Choose a base branch
from
Open

Conversation

martinkilbinger
Copy link
Contributor

@martinkilbinger martinkilbinger commented Jan 15, 2025

Summary

Major modifications for processing run on canfar of ShapePipe v1.4 catalogues.

Auxilliary scripts:

  • Added bash scripts for job handling on new canfar science portal (curl based)
  • New python script to create final catalogue as hdf5 file.

Main pipeline:

  • New command line option-e ID to select exclusive image ID
  • Created new docker image, to replace conda installation

Modules:

  • Batch saving for ngmix; output catalogue can be written to disk in batches. In case the (typically very time-consuming)
    processing is interrupted, reprocessing does not have to start from the beginning. Note that in this case the random numbers
    do not match the previous object ID any more.
  • Added check_existing_dir option to modules mask, ngmix. Processing of the corresponding module is skipped if
    expected output file exists in this directory.
  • Added file tests to many modules, improved error messages
  • make catalogue runner: spread model input now optional
  • merged star cat output moved from HDU Merge pipeline template to master branch #2 to HDU test ci setup #1; added HDU config option for input

Reviewer Checklist

Reviewers should tick the following boxes before approving and merging the PR.

  • The PR targets the develop branch
  • The PR is assigned to the developer
  • The PR has appropriate labels
  • The PR is included in appropriate projects and/or milestones
  • The PR includes a clear description of the proposed changes
  • If the PR addresses an open issue the description includes "closes #"
  • The code and documentation style match the current standards
  • Documentation has been added/updated consistently with the code
  • All CI tests are passing
  • API docs have been built and checked at least once (if relevant)
  • All changed files have been checked and comments provided to the developer
  • All of the reviewer's comments have been satisfactorily addressed by the developer

remove 000.000
Mask runner skip processing if output file exists
@martinkilbinger martinkilbinger added the enhancement New feature or request label Jan 15, 2025
@martinkilbinger martinkilbinger added this to the ShapePipe v1.4 milestone Jan 15, 2025
@martinkilbinger martinkilbinger self-assigned this Jan 15, 2025
shape1_cat_path,
) = input_file_list
sexcat_sm_path = None
else:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New: spread model input is optional

sexcat_sm_path,
SEx_catalogue=True,
)
sexcat_sm_file.open()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

spread model input is not optional

@@ -1180,7 +1188,15 @@ def _build_final_mask(
external_flag.open()
if final_mask is not None:
final_mask = final_mask.astype(np.int16, copy=False)
final_mask += external_flag.get_data()[:, :]
try:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added file check

open_mode=file_io.BaseCatalogue.OpenMode.ReadWrite,
SEx_catalogue=True
SEx_catalogue=False,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Propagating information from original catalogue (True) is not needed, and moves the main data to HDU #2, not compatible with other files.

@martinkilbinger martinkilbinger changed the base branch from develop to develop_black January 15, 2025 10:25
@martinkilbinger martinkilbinger changed the base branch from develop_black to develop January 15, 2025 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: To do
Development

Successfully merging this pull request may close these issues.

2 participants