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

First work on paginated tensors. #81

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

First work on paginated tensors. #81

wants to merge 2 commits into from

Conversation

chewxy
Copy link
Member

@chewxy chewxy commented Oct 12, 2020

No description provided.

Co-authored-by: Jeremy Khawaja <[email protected]>
@ghost
Copy link

ghost commented Oct 12, 2020

Thanks for opening this one up @chewxy!

I guess we want to integrate the paginated code following the existing file structure e.g., paginated_format.go, paginated_arith.go. Likely getting the code to follow closely to the Dense type.

Or do we want to try to integrate pagination as an option on a Dense tensor? It would probably just blow up the amount of code inside the Dense files.

EDIT: or we can just move everything into a new folder like you started.

@ghost ghost mentioned this pull request Oct 13, 2020
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.

1 participant