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-13876: implement ParquetStorage butler storage mechanism #89

Merged
merged 3 commits into from May 1, 2018

Conversation

timothydmorton
Copy link
Contributor

This changes the storage type and dataset type of deepCoadd_obj to use the new implementation of ParquetStorage and ParquetTable.

Change the storage type from FitsStorage to ParquetStorage,
which this ticket implements, and change the python datatype
to the new ParquetTable datatype, which is for now defined
within qa_explorer.
@@ -806,14 +806,15 @@ deepCoadd_obj: #consolidated multi-band object tables from coadd
description: >
Consolidated coadd multi-band object table, a merge of
deepCoadd_meas, deepCoadd_forced_src and deepCoadd_ref tables for multiple bands.
This is a pandas DataFrame with a multi-level column index; access a particular
subcatalog as df['meas']['HSC-R'], e.g.
This is stored as a DataFrame with a multi-level column index; access a particular
Copy link
Contributor

Choose a reason for hiding this comment

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

DataFrame -> ParquetTable?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not really; the comment here regards the fact that it is actually a multilevel pandas DataFrame that is stored in the parquet file.

storage: FitsCatalogStorage
python: lsst.qa.explorer.table.ParquetTable
storage: ParquetStorage
python: lsst.qa.explorer.parquetTable.MultilevelParquetTable # pandas.DataFrame to write
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove comment?

@timothydmorton timothydmorton merged commit 0cd0e28 into master May 1, 2018
@ktlim ktlim deleted the tickets/DM-13876 branch August 25, 2018 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants