Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[python] Fix write from array that is not row or column order #2542

Merged
merged 1 commit into from
May 10, 2024

Conversation

jp-dark
Copy link
Collaborator

@jp-dark jp-dark commented May 9, 2024

Issue and/or context: #2537

Changes:
When writing a dense array, this change will check if the data is neither row major nor column major and explicitly set it to be row-major before writing.

@jp-dark jp-dark requested a review from nguyenv May 9, 2024 20:55
@jp-dark
Copy link
Collaborator Author

jp-dark commented May 9, 2024

Closes #2537

@jp-dark jp-dark changed the title Fix write from array that is not row or column order [python] Fix write from array that is not row or column order May 9, 2024
Copy link

codecov bot commented May 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.21%. Comparing base (d8cac0b) to head (558fac2).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2542   +/-   ##
=======================================
  Coverage   90.20%   90.21%           
=======================================
  Files          37       37           
  Lines        3971     3975    +4     
=======================================
+ Hits         3582     3586    +4     
  Misses        389      389           
Flag Coverage Δ
python 90.21% <100.00%> (+<0.01%) ⬆️

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

Components Coverage Δ
python_api 90.21% <100.00%> (+<0.01%) ⬆️
libtiledbsoma ∅ <ø> (∅)

Copy link
Member

@nguyenv nguyenv left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks so much for looking at this.

@jp-dark jp-dark merged commit 204f0c8 into main May 10, 2024
11 checks passed
@jp-dark jp-dark deleted the dark/fix-regression branch May 10, 2024 13: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