Skip to content

Allow nested column names in statistics methods#625

Merged
camposandro merged 2 commits intomainfrom
stats-nested-cols
Jan 15, 2026
Merged

Allow nested column names in statistics methods#625
camposandro merged 2 commits intomainfrom
stats-nested-cols

Conversation

@camposandro
Copy link
Copy Markdown
Contributor

Allow nested column names in include_columns and exclude_columns, for per_pixel_statistics and aggregate_column_statistics. For example, if "lightcurve" is a nested column, we can now specify include_columns=["lightcurve"] as well as exclude_columns=["lightcurve"].

@camposandro camposandro self-assigned this Jan 14, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.49%. Comparing base (6081879) to head (a2610ef).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #625      +/-   ##
==========================================
+ Coverage   93.07%   93.49%   +0.42%     
==========================================
  Files          50       50              
  Lines        2367     2444      +77     
==========================================
+ Hits         2203     2285      +82     
+ Misses        164      159       -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.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 14, 2026

Before [cce3f3a] After [c4ce459] Ratio Benchmark (Parameter)
1.04±0.02ms 1.06±0.02ms 1.02 benchmarks.time_test_cone_filter_multiple_order
12.9±0.5ms 13.0±0.5ms 1.01 benchmarks.Suite.time_inner_pixel_alignment
400±4ms 404±1ms 1.01 benchmarks.Suite.time_outer_pixel_alignment
270±3ms 272±0.6ms 1.01 benchmarks.time_open_large_catalog
41.9±0.4ms 41.7±0.6ms 0.99 benchmarks.Suite.time_pixel_tree_creation
43.8±0.2ms 43.5±0.2ms 0.99 benchmarks.time_open_midsize_catalog
277±2ms 273±2ms 0.99 benchmarks.time_small_cone_large_catalog
123±2ms 123±0.7ms 0.99 benchmarks.time_test_alignment_even_sky

Click here to view all benchmarks.

@camposandro camposandro merged commit e34ae82 into main Jan 15, 2026
12 checks passed
@camposandro camposandro deleted the stats-nested-cols branch January 15, 2026 14:28
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