Skip to content

Deep copying MPS with copy#387

Merged
lkdvos merged 5 commits intoQuantumKitHub:mainfrom
20akshay00:mps-copy
Feb 23, 2026
Merged

Deep copying MPS with copy#387
lkdvos merged 5 commits intoQuantumKitHub:mainfrom
20akshay00:mps-copy

Conversation

@20akshay00
Copy link
Copy Markdown
Contributor

Fixes #386. Probably should add more tests also for copy operations on all other objects like MPOs.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 17, 2026

Your PR no longer requires formatting changes. Thank you for your contribution!

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/states/finitemps.jl 94.75% <100.00%> (+0.16%) ⬆️
src/states/infinitemps.jl 79.41% <100.00%> (+2.94%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Member

@lkdvos lkdvos left a comment

Choose a reason for hiding this comment

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

Except for the formatting issues, this looks great, thanks!
As a small FYI, I slightly screwed up the last PR so still have to fix some of the tests, so failures might actually be unrelate

@lkdvos lkdvos enabled auto-merge (squash) February 22, 2026 22:56
@lkdvos lkdvos disabled auto-merge February 23, 2026 16:58
@lkdvos lkdvos merged commit 2220cd1 into QuantumKitHub:main Feb 23, 2026
32 of 34 checks passed
lkdvos added a commit that referenced this pull request Feb 23, 2026
* broadcast copy over arrays

* add basic tests for copying mps

* fix variable naming mismatch

* small fixes

* format

---------

Co-authored-by: Lukas Devos <ldevos98@gmail.com>
lkdvos added a commit that referenced this pull request Feb 24, 2026
* broadcast copy over arrays

* add basic tests for copying mps

* fix variable naming mismatch

* small fixes

* format

---------

Co-authored-by: Lukas Devos <ldevos98@gmail.com>
lkdvos added a commit that referenced this pull request Feb 24, 2026
* broadcast copy over arrays

* add basic tests for copying mps

* fix variable naming mismatch

* small fixes

* format

---------

Co-authored-by: Lukas Devos <ldevos98@gmail.com>
lkdvos added a commit that referenced this pull request Feb 24, 2026
* broadcast copy over arrays

* add basic tests for copying mps

* fix variable naming mismatch

* small fixes

* format

---------

Co-authored-by: Lukas Devos <ldevos98@gmail.com>
lkdvos added a commit that referenced this pull request Feb 24, 2026
* Revert "Adapt extension (#384)"

This reverts commit 2db4cb1.

* Deep copying MPS with copy (#387)

* broadcast copy over arrays

* add basic tests for copying mps

* fix variable naming mismatch

* small fixes

* format

---------

Co-authored-by: Lukas Devos <ldevos98@gmail.com>

* Refactor entropy function to also use spectrum directly (#377)

* Refactor entropy function to also use spectrum directly

* specialize to SectorVector and add test

* add infinite test

* update docstring

---------

Co-authored-by: Lukas Devos <ldevos98@gmail.com>

* Deep copying MPS with copy (#387)

* broadcast copy over arrays

* add basic tests for copying mps

* fix variable naming mismatch

* small fixes

* format

---------

Co-authored-by: Lukas Devos <ldevos98@gmail.com>

* fix wrong testcase

* alter JordanMPOTensor Adapt implementation

* type stability chenanigans

* bump TensorKit minimal version

* more bypassing of type stability issues...

* fix some tests

* lost of git struggles

* more git misery!

* handle storagetype better

* disable lts type stability tests

---------

Co-authored-by: Akshay Shankar <sakshays.2000@gmail.com>
Co-authored-by: Andreas Feuerpfeil <andreas.feuerpfeil@gmail.com>
@lkdvos lkdvos mentioned this pull request Feb 26, 2026
lkdvos referenced this pull request Feb 26, 2026
* update compat to remove broken versions

* update changelog

* update citation file

* update changelog for release
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.

Copying an MPS currently only makes a shallow copy

2 participants