Skip to content

Conversation

@delucchi-cmu
Copy link
Contributor

Closes #568

Generates VOTable metadata for a catalog, given the existing names and dtypes in the _common_metadata. Additional method to write this metadata back to the _common_metadata file in VOParquet key-value pairs.

@delucchi-cmu delucchi-cmu marked this pull request as ready for review November 4, 2025 18:36
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

Before [d5fdcaa] <v0.7.0> After [fe97067] Ratio Benchmark (Parameter)
390±2ms 403±5ms 1.04 benchmarks.Suite.time_outer_pixel_alignment
265±4ms 267±2ms 1.01 benchmarks.time_open_large_catalog
125±0.3ms 126±1ms 1.01 benchmarks.time_test_alignment_even_sky
12.5±0.4ms 12.5±0.3ms 1 benchmarks.Suite.time_inner_pixel_alignment
41.0±0.9ms 41.1±1ms 1 benchmarks.Suite.time_pixel_tree_creation
269±0.6ms 269±0.7ms 1 benchmarks.time_small_cone_large_catalog
1.03±0.03ms 1.02±0.04ms 0.99 benchmarks.time_test_cone_filter_multiple_order
42.5±0.2ms 41.8±0.4ms 0.98 benchmarks.time_open_midsize_catalog

Click here to view all benchmarks.

@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

❌ Patch coverage is 98.82353% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 93.50%. Comparing base (d5fdcaa) to head (f87125a).

Files with missing lines Patch % Lines
src/hats/io/parquet_metadata.py 98.78% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #591      +/-   ##
==========================================
+ Coverage   93.05%   93.50%   +0.44%     
==========================================
  Files          48       48              
  Lines        2145     2216      +71     
==========================================
+ Hits         1996     2072      +76     
+ Misses        149      144       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

Very Basic VOParquet metadata generation

2 participants