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

Add a partition_info on SOAP results. #157

Merged
merged 1 commit into from
Nov 1, 2023
Merged

Conversation

delucchi-cmu
Copy link
Contributor

Change Description

This is a little silly. Even though we'd like to remove partition_info.csv and partition_join_info.csv, they're required with the most recent version of the hipscat Dataset class. We need to be a valid dataset so we can release a new version of the hipscat library and still pass builds here.

@delucchi-cmu delucchi-cmu merged commit 7038f7d into main Nov 1, 2023
11 checks passed
@delucchi-cmu delucchi-cmu deleted the delucchi/soap_info branch November 2, 2023 16:56
delucchi-cmu added a commit that referenced this pull request Nov 6, 2023
* Set lower default import order. (#153)

* Macauff pipeline boilerplate (#152)

* checkpoint

* checkpoint

* testing for MacauffArguments

* create boilerplate for macauff runner + tests

* remove commented out test

* add __future__.annotations

* linter problems

* add more tests for missing coverage

* refactor MacauffArguments required parameter tests

* address more comments from pr #152

* add dask pytest mark + black formatting

* Add a partition_info on SOAP results. (#157)

* Initial commit of metadata file conversion (#156)

* Initial commit of metadata file conversion

* Move shared logic for locating input files.

* Explicitly depend on pyyaml

---------

Co-authored-by: Max West <[email protected]>
delucchi-cmu added a commit that referenced this pull request Nov 6, 2023
* Use existing hipscat_index for mapping pixel

* Update required version.

* Merge recent changes (#163)

* Set lower default import order. (#153)

* Macauff pipeline boilerplate (#152)

* checkpoint

* checkpoint

* testing for MacauffArguments

* create boilerplate for macauff runner + tests

* remove commented out test

* add __future__.annotations

* linter problems

* add more tests for missing coverage

* refactor MacauffArguments required parameter tests

* address more comments from pr #152

* add dask pytest mark + black formatting

* Add a partition_info on SOAP results. (#157)

* Initial commit of metadata file conversion (#156)

* Initial commit of metadata file conversion

* Move shared logic for locating input files.

* Explicitly depend on pyyaml

---------

Co-authored-by: Max West <[email protected]>

* Exercise more hipscat_index behavior.

---------

Co-authored-by: Max West <[email protected]>
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