Patch fixes for v1.0.1 #589
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* updating job script and tutorial * tutorial; fix config file * tutorial finished * no change * psfex update (minor bug) * psfex running * updated MCCD config files * comment in config file * config file fix for MCCD * config file fix for MCCD * removed unused option in usage str * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * improved tutorial according to PR review * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * tutorial clean-up * post_proc cleaned up; tutorial added output dir names * changes to tutorial, post-proc and mccd plot doc * get_images error msg testing * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * removed prompt from bash example * Update docs/source/post_processing.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * PR review changes to doc * PR review changes to doc * Update docs/source/post_processing.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com>
* added contribution guidelines * Update CODE_OF_CONDUCT.md
Co-authored-by: Lucie Baumont <lucie.baumont@cea.fr>
* added md for understanding the api docs * updates following reviwer comments
* updated doc-tests * updated package versions * pin documentation packages * updated ShapePipe version and cleaned up
* Removed unused config files in example/cfis * Removed obsolete config dirs tutorial and ps3pi_cfis * removed old unions config files
Co-authored-by: Martin Kilbinger <martin.kilbinger@cea.fr>
* merge_final script: added option for additional tile ID path to select tiles; clean-up * typo in doc string
removed errornous mccd_interp_runner call
* updated docs and references * Update docs/source/module_develop.md Co-authored-by: Martin Kilbinger <martin.kilbinger@cea.fr> Co-authored-by: Martin Kilbinger <martin.kilbinger@cea.fr>
* merge_final script: added option for additional tile ID path to select tiles; clean-up * typo in doc string * started to implement healpix mask output * healpix options for random/mask * Update shapepipe/modules/random_cat_package/random_cat.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/random_cat_package/random_cat.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/random_cat_package/random_cat.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/random_cat_package/random_cat.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Modifications for PR review * comments from PR review Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com>
* added masking for NGC objects * debugging ngc np binary file * Fixed bugs in new mask handling (input FITS files) * debug info to log file for checking existing images * mask: combined dso masks into dictionary * PR edits * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * added symlink * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * PR suggestions Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com>
* random runner: hp masks working; doc updated * random runner config file updated * Update shapepipe/modules/random_cat_package/random_cat.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * passing tests * Update shapepipe/modules/random_cat_package/__init__.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/random_cat_package/__init__.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com>
* Summer 2022 Patch Release v1.0.1 (#581) * added CD GitHub action * fixed bug in RunLog class (#548) * 800deg2 (#544) * added config files for matching 800deg2 survey * Added Xavier as contributor; match external catalogue fix * match 800 * match_ext: output distance * match_ext output_dist def value * Tutorial documentation (#543) * updating job script and tutorial * tutorial; fix config file * tutorial finished * no change * psfex update (minor bug) * psfex running * updated MCCD config files * comment in config file * config file fix for MCCD * config file fix for MCCD * removed unused option in usage str * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * improved tutorial according to PR review * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * tutorial clean-up * post_proc cleaned up; tutorial added output dir names * changes to tutorial, post-proc and mccd plot doc * get_images error msg testing * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update docs/source/pipeline_tutorial.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * removed prompt from bash example * Update docs/source/post_processing.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * PR review changes to doc * PR review changes to doc * Update docs/source/post_processing.md Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * added contribution guidelines (#550) * added contribution guidelines * Update CODE_OF_CONDUCT.md * tutorial bugfixes (#552) Co-authored-by: Lucie Baumont <lucie.baumont@cea.fr> * removed pep8 error exceptions (#554) * Understanding the API docs (#553) * added md for understanding the api docs * updates following reviwer comments * Update package versions (#539) * updated doc-tests * updated package versions * pin documentation packages * updated ShapePipe version and cleaned up * Config file clean up (#556) * Removed unused config files in example/cfis * Removed obsolete config dirs tutorial and ps3pi_cfis * removed old unions config files * Update docs/source/configuration.md Co-authored-by: Martin Kilbinger <martin.kilbinger@cea.fr> * final clean up * removed obsolete distance-based overlap region flagging * removed errornous mccd_interp_runner call * Added Tobias' MCCD plot config file to validate joint cat * Added Tobias' MCCD plot config file to validate joint cat * removed duplicate config file * Random catalogue improvements (#570) * merge_final script: added option for additional tile ID path to select tiles; clean-up * typo in doc string * Updated docs and references (#579) * updated docs and references * Update docs/source/module_develop.md Co-authored-by: Martin Kilbinger <martin.kilbinger@cea.fr> Co-authored-by: Martin Kilbinger <martin.kilbinger@cea.fr> * Random runner updates (#573) * merge_final script: added option for additional tile ID path to select tiles; clean-up * typo in doc string * started to implement healpix mask output * healpix options for random/mask * Update shapepipe/modules/random_cat_package/random_cat.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/random_cat_package/random_cat.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/random_cat_package/random_cat.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/random_cat_package/random_cat.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Modifications for PR review * comments from PR review Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * added masking for NGC objects (#575) * added masking for NGC objects * debugging ngc np binary file * Fixed bugs in new mask handling (input FITS files) * debug info to log file for checking existing images * mask: combined dso masks into dictionary * PR edits * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * added symlink * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/modules/mask_package/mask.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * Update shapepipe/pipeline/file_io.py Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * PR suggestions Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> * update patch version Co-authored-by: Martin Kilbinger <martin.kilbinger@cea.fr> Co-authored-by: Lucie Baumont <lucie.baumont@stonybrook.edu> Co-authored-by: Lucie Baumont <lucie.baumont@cea.fr> Co-authored-by: Tobias Liaudat <tobiasliaudat@gmail.com> * fixed typo in variable name * config file vignet runner output fixed * typo in make_cat config ini file fixed Co-authored-by: Samuel Farrens <samuel.farrens@gmail.com> Co-authored-by: Lucie Baumont <lucie.baumont@stonybrook.edu> Co-authored-by: Lucie Baumont <lucie.baumont@cea.fr> Co-authored-by: Tobias Liaudat <tobiasliaudat@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This patch revision includes the following PRs: