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

[c++] Integrate SOMAColumn: Update domain accessors inside SOMAArray, part 2 #3407

Open
wants to merge 6 commits into
base: xan/sc-59427/soma-column-arrow-integration
Choose a base branch
from

Conversation

XanthosXanthopoulos
Copy link
Collaborator

During the SOMAArray object construction and open the SOMAColumn objects are initialized and used througout the lifetime of the SOMAArray object to access the different domain types for the object. The complexity of transforming the data to an ArrowTable has been transferred to each SOMAColumn and that makes SOMAArray implementation agnostic of any column specific restrictions.

@XanthosXanthopoulos XanthosXanthopoulos changed the title [c++] Use SOMAColumn to access domain inside SOMAArray Part 3 [c++] Use SOMAColumn to access domain inside SOMAArray Part 3 [WIP] Dec 9, 2024
@XanthosXanthopoulos XanthosXanthopoulos marked this pull request as draft December 9, 2024 08:48
@johnkerl johnkerl changed the title [c++] Use SOMAColumn to access domain inside SOMAArray Part 3 [WIP] [c++] Use SOMAColumn to access domain inside SOMAArray part 3 [WIP] Dec 9, 2024
@XanthosXanthopoulos XanthosXanthopoulos force-pushed the xan/sc-59427/soma-column-somarray-integration branch from cbe3cf0 to 29448a2 Compare December 10, 2024 11:16
@XanthosXanthopoulos XanthosXanthopoulos marked this pull request as ready for review December 10, 2024 16:10
@XanthosXanthopoulos XanthosXanthopoulos force-pushed the xan/sc-59427/soma-column-arrow-integration branch from b1bd03c to 5485141 Compare December 11, 2024 11:44
@XanthosXanthopoulos XanthosXanthopoulos force-pushed the xan/sc-59427/soma-column-somarray-integration branch from 29448a2 to d4e4d7e Compare December 11, 2024 11:45
@XanthosXanthopoulos XanthosXanthopoulos force-pushed the xan/sc-59427/soma-column-arrow-integration branch 2 times, most recently from 0e69ed7 to d6d6187 Compare December 13, 2024 16:10
@XanthosXanthopoulos XanthosXanthopoulos force-pushed the xan/sc-59427/soma-column-somarray-integration branch from d4e4d7e to e746f3b Compare December 13, 2024 16:17
@XanthosXanthopoulos XanthosXanthopoulos changed the title [c++] Use SOMAColumn to access domain inside SOMAArray part 3 [WIP] [c++] Integrate SOMAColumn: Update domain accessors inside SOMAArray, part 2 Dec 13, 2024
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.

[c++] Add an abstraction layer between SOMA columns and TileDB dimensions and attributes
1 participant