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

Add index_base as optional argument to export_data #315

Open
jeremy-myers opened this issue Sep 19, 2024 · 0 comments
Open

Add index_base as optional argument to export_data #315

jeremy-myers opened this issue Sep 19, 2024 · 0 comments
Labels
capability New capability/functionality

Comments

@jeremy-myers
Copy link
Collaborator

export_data currently writes out sparse index sets in 1-based indexing, but import_data allows any arbitrary base. It's not obvious that we enforce one base on write but allow any on read.

for i in range(A.nnz):
    # 0-based indexing in package, 1-based indexing in file
    subs = A.subs[i, :] + 1 # <- Here we assume 1-based indexing
@jeremy-myers jeremy-myers added the capability New capability/functionality label Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
capability New capability/functionality
Projects
None yet
Development

No branches or pull requests

1 participant