Skip to content

Hdf5#783

Draft
manuschneider wants to merge 48 commits into
masterfrom
hdf5
Draft

Hdf5#783
manuschneider wants to merge 48 commits into
masterfrom
hdf5

Conversation

@manuschneider
Copy link
Copy Markdown
Collaborator

Implements HDF5 support for saving and loading

added const attribute to operator+-*/% for Tensor
@manuschneider manuschneider marked this pull request as ready for review May 2, 2026 20:16
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 81d4c5358d

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/backend/Storage.cpp Outdated
Comment thread src/Tensor.cpp Outdated
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4242b9996a

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread CMakeLists.txt Outdated
Comment thread src/BlockUniTensor.cpp Outdated
Comment thread src/BlockFermionicUniTensor.cpp Outdated
Comment thread src/Bond.cpp Outdated
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ec5b97f4db

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/DenseUniTensor.cpp Outdated
Comment thread src/BlockUniTensor.cpp Outdated
Comment thread src/BlockFermionicUniTensor.cpp Outdated
Comment thread src/Bond.cpp Outdated
@manuschneider
Copy link
Copy Markdown
Collaborator Author

The API uses the existing Save and Load methods for HDF5 files.
Unfortunately, passing open HDF5 files to/from python is not easily possible. Therefore, each Save/Load opens the file and closes it again. To keep the file open, the methods to_hdf5 and from_hdf5 can be used, but only in C++.

Feel free to comment on the APIs and the data format

@manuschneider manuschneider force-pushed the hdf5 branch 6 times, most recently from 7765473 to efff52c Compare May 5, 2026 08:54
@manuschneider manuschneider mentioned this pull request May 30, 2026
2 tasks
@pcchen pcchen added this to the v1.2.0 milestone Jun 1, 2026
@pcchen
Copy link
Copy Markdown
Collaborator

pcchen commented Jun 1, 2026

I will take this PR with milestone v1.2.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants