Skip to content

[python] Add bindings for ndarrays#2129

Merged
nguyenv merged 1 commit intomainfrom
viviannguyen/add-pybind11-ndarray-bindings
Feb 15, 2024
Merged

[python] Add bindings for ndarrays#2129
nguyenv merged 1 commit intomainfrom
viviannguyen/add-pybind11-ndarray-bindings

Conversation

@nguyenv
Copy link
Copy Markdown
Contributor

@nguyenv nguyenv commented Feb 12, 2024

Issue and/or context:

#2128

Changes:

Addition of soma_dense_ndarray.cc and soma_sparse.ndarray.cc.

These changes were picked out of #1817.

Notes for Reviewer:

These changes are on top of #2126.

@nguyenv nguyenv linked an issue Feb 12, 2024 that may be closed by this pull request
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 12, 2024

Codecov Report

Merging #2129 (5fe06d0) into main (a4f68a8) will decrease coverage by 3.44%.
The diff coverage is 0.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2129      +/-   ##
==========================================
- Coverage   82.10%   78.67%   -3.44%     
==========================================
  Files          87      136      +49     
  Lines        8112    10722    +2610     
  Branches        0      211     +211     
==========================================
+ Hits         6660     8435    +1775     
- Misses       1452     2185     +733     
- Partials        0      102     +102     
Flag Coverage Δ
libtiledbsoma 66.84% <0.00%> (?)
python 91.35% <ø> (+0.64%) ⬆️
r 74.62% <ø> (ø)

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

Components Coverage Δ
python_api 91.35% <ø> (+0.64%) ⬆️
libtiledbsoma 48.05% <0.00%> (∅)

Base automatically changed from viviannguyen/refactor-pybind11-inheritance to main February 12, 2024 21:42
@nguyenv nguyenv force-pushed the viviannguyen/add-pybind11-ndarray-bindings branch from 5b1a40b to 5fe06d0 Compare February 13, 2024 04:28
@nguyenv nguyenv requested review from gspowley and johnkerl February 13, 2024 04:31
Copy link
Copy Markdown
Member

@gspowley gspowley left a comment

Choose a reason for hiding this comment

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

LGTM!

@nguyenv nguyenv merged commit 3bfb289 into main Feb 15, 2024
@nguyenv nguyenv deleted the viviannguyen/add-pybind11-ndarray-bindings branch February 15, 2024 19:15
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.

[python] Add Sparse and Dense NDArray bindings

2 participants