Skip to content

Remove deprecated "mlp" and "transformers" network name aliases - #197

Merged
hzhangxyz merged 2 commits into
mainfrom
copilot/remove-deprecated-model-alias
Mar 5, 2026
Merged

Remove deprecated "mlp" and "transformers" network name aliases#197
hzhangxyz merged 2 commits into
mainfrom
copilot/remove-deprecated-model-alias

Conversation

Copilot AI commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

Compatibility aliases "mlp" and "transformers" allowed users to omit symmetry from network names, but this implicit default is now removed. All networks must explicitly declare their symmetry.

Changes

  • hubbard.py — removed "mlp" (→ "mlp/u1u1") and "transformers" (→ "transformers/u1u1") aliases
  • ising.py — removed "mlp" (→ "mlp/0") and "transformers" (→ "transformers/0") aliases
  • fcidump.py, openfermion.py, optimized_basis.py — same removals as hubbard.py

Migration

# Before (no longer valid)
network:
  name: mlp

# After
network:
  name: mlp/u1u1   # or mlp/u1, mlp/0, transformers/u1u1, etc.
Original prompt

This section details on the original issue you should resolve

<issue_title>删除过时的模型alias</issue_title>
<issue_description>models文件夹中, 比如hubbard, mlp/u1u1 又名 mlp, 但这个只是为了兼容, 现在删除mlp, 所有的网络必explicit表示其对称性</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: hzhangxyz <11623447+hzhangxyz@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove deprecated model alias for mlp Remove deprecated "mlp" and "transformers" network name aliases Mar 5, 2026
@hzhangxyz
hzhangxyz marked this pull request as ready for review March 5, 2026 19:27
Copilot AI review requested due to automatic review settings March 5, 2026 19:27
@hzhangxyz
hzhangxyz merged commit 28a96d2 into main Mar 5, 2026
8 checks passed
Copilot AI removed the request for review from Copilot March 23, 2026 22:43
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.

删除过时的模型alias

2 participants