Skip to content

Reimplement flow matching using Gaussian elimination (local merge)#51

Merged
HaoTy merged 17 commits into
mainfrom
enhance/flow-matching
May 3, 2026
Merged

Reimplement flow matching using Gaussian elimination (local merge)#51
HaoTy merged 17 commits into
mainfrom
enhance/flow-matching

Conversation

@HaoTy

@HaoTy HaoTy commented Apr 11, 2026

Copy link
Copy Markdown
Collaborator

This is a local version of #45 to avoid GitHub action permission issues when merging from a fork.

@smburdick

Copy link
Copy Markdown
Contributor

@HaoTy is this ready to be reviewed?

@HaoTy

HaoTy commented Apr 17, 2026

Copy link
Copy Markdown
Collaborator Author

@HaoTy is this ready to be reviewed?

It's the same version of #45 that you reviewed.

@smburdick smburdick left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Copilot <copilot@github.com>
Comment thread src/tqecd/cover.py Outdated
Comment thread src/tqecd/cover.py Outdated
Comment thread src/tqecd/cover.py
HaoTy added 2 commits May 2, 2026 02:24
…the source indices even if sources are not completely linearly independent
@HaoTy HaoTy requested a review from inmzhang May 2, 2026 07:58

@inmzhang inmzhang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks for the work!

Do you think we can release a v0.2.0 of tqecd after this PR is merged? Feel free to ask me to push the tag if you have no permission to do that.

@HaoTy

HaoTy commented May 3, 2026

Copy link
Copy Markdown
Collaborator Author

LGTM, thanks for the work!

Do you think we can release a v0.2.0 of tqecd after this PR is merged? Feel free to ask me to push the tag if you have no permission to do that.

The current version change in pyproject.toml is from 0.1.3 to 0.1.4. But it does make sense to do 0.2.0. Let me update pyproject.toml.

@github-actions

github-actions Bot commented May 3, 2026

Copy link
Copy Markdown

Code Coverage

Package Line Rate Complexity Health
src.tqecd 94% 0
Summary 94% (756 / 808) 0

@HaoTy HaoTy merged commit f21e9ed into main May 3, 2026
8 checks passed
@HaoTy HaoTy deleted the enhance/flow-matching branch May 3, 2026 04:27
@github-actions

github-actions Bot commented May 3, 2026

Copy link
Copy Markdown

🪓 PR closed, deleted preview at https://tqec.github.io/tqecd/pull/51/

@HaoTy

HaoTy commented May 3, 2026

Copy link
Copy Markdown
Collaborator Author

Just pushed a tag to main. Looks like the PyPI publish workflow is running. Thanks!

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.

3 participants