Skip to content

Adding shape validation for add_matrix_to_collection (#4258)#4329

Merged
jp-dark merged 1 commit intorelease-2.2from
backport-4258-to-release-2.2
Dec 5, 2025
Merged

Adding shape validation for add_matrix_to_collection (#4258)#4329
jp-dark merged 1 commit intorelease-2.2from
backport-4258-to-release-2.2

Conversation

@jp-dark
Copy link
Copy Markdown
Collaborator

@jp-dark jp-dark commented Dec 5, 2025

Backport PR #4258

* Adding shape validation for add_matrix_to_collection

* Adding test for add_X_layer

* Add context to the validation function

* Schema validation optional default False

* PR fixes

* PR fixes vol 2

* Adding all versions in tests

* Adding full docstrings

* Add entry in HISTORY.md

* Minor fixes for PR

* Opening internal var dataframe in read mode when writing & text fix

* PR fixes - timestamp, closed property

* Add unit tests for previous versions

* PR comments

* PR comments

* Add warning instead of log msg

* Rebase with main

* Final misc

* Match argument to all tests

(cherry picked from commit 2d91da6)
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 5, 2025

Codecov Report

❌ Patch coverage is 96.96970% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 89.45%. Comparing base (1753ab9) to head (01b00da).
⚠️ Report is 16 commits behind head on release-2.2.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-2.2    #4329       +/-   ##
================================================
+ Coverage        76.13%   89.45%   +13.31%     
================================================
  Files              233       61      -172     
  Lines            32559     7176    -25383     
  Branches          1236        0     -1236     
================================================
- Hits             24789     6419    -18370     
+ Misses            7346      757     -6589     
+ Partials           424        0      -424     
Flag Coverage Δ
libtiledbsoma ?
python 89.45% <96.96%> (+<0.01%) ⬆️
r ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
python_api 89.45% <96.96%> (+<0.01%) ⬆️
libtiledbsoma ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jp-dark jp-dark merged commit b707302 into release-2.2 Dec 5, 2025
11 of 12 checks passed
@jp-dark jp-dark deleted the backport-4258-to-release-2.2 branch December 5, 2025 19:55
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