Skip to content

More update for numpy 2.0 casting changes in product __add__ - #439

Merged
nicHoch merged 1 commit into
i4Ds:masterfrom
samaloney:bugfix-py311-packages
Nov 15, 2025
Merged

nicHoch merged 1 commit into
i4Ds:masterfrom
samaloney:bugfix-py311-packages

Conversation

@samaloney

Copy link
Copy Markdown
Collaborator

When adding two products need to ensure the index dtype column is sufficient to hold the combine number of rows.

@codecov-commenter

codecov-commenter commented Nov 11, 2025 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.36364% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.15%. Comparing base (3ae9671) to head (03db081).
⚠️ Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
stixcore/products/product.py 60.00% 2 Missing ⚠️
stixcore/products/level0/quicklookL0.py 88.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #439      +/-   ##
==========================================
- Coverage   72.23%   69.15%   -3.08%     
==========================================
  Files          78       76       -2     
  Lines        8085     8090       +5     
==========================================
- Hits         5840     5595     -245     
- Misses       2245     2495     +250     

☔ 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.

@samaloney
samaloney force-pushed the bugfix-py311-packages branch from c9048fd to 03db081 Compare November 14, 2025 20:10

@nicHoch nicHoch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good to go

data["timedel"] = duration
data.add_meta(name="timedel", nix="NIX00405", packets=packets)
data["triggers"] = triggers.astype(get_min_uint(triggers))
data["triggers"] = triggers.astype(np.min_scalar_type(triggers))

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nice that this is common numpy now

@nicHoch
nicHoch merged commit a2bb3d9 into i4Ds:master Nov 15, 2025
8 of 9 checks passed
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.

3 participants