Skip to content

Rename subcommand_dictaction_dict and SubcommandProtoActionProto - #193

Merged
hzhangxyz merged 3 commits into
mainfrom
copilot/rename-subcommand-to-action
Mar 5, 2026
Merged

Rename subcommand_dictaction_dict and SubcommandProtoActionProto#193
hzhangxyz merged 3 commits into
mainfrom
copilot/rename-subcommand-to-action

Conversation

Copilot AI commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

The interface moved from argv-based subcommands to YAML action parameters, but the internal naming still used "subcommand" terminology throughout. This renames all relevant symbols, files, and comments to reflect the current interface.

Changes

  • qmp/utility/subcommand_dict.pyqmp/utility/action_dict.py: Renamed module; SubcommandProtoActionProto, subcommand_dictaction_dict
  • All algorithm files (vmc, haar, orbit, guide, pert, chop_imag, haar_with_orbit, pretrain): Updated imports and registration calls
  • qmp/__main__.py: Updated import and lookup via action_dict
  • Deprecation warnings in haar.py and haar_with_orbit.py: "The 'imag' subcommand is deprecated...""The 'imag' action is deprecated..."

Example

Before:

from qmp.utility.subcommand_dict import subcommand_dict
subcommand_dict["vmc"] = VmcConfig

After:

from qmp.utility.action_dict import action_dict
action_dict["vmc"] = VmcConfig
Original prompt

This section details on the original issue you should resolve

<issue_title>所有的 "subcommand"应该替换成 "action"</issue_title>
<issue_description>因为使用接口改变了, 不再是argv的形式, 而是通过yaml里的参数action实现, 仓库中的subcommand_dict应该rename为action_dict, 其他诸如注释中的subcommand词也应该更新</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.

Copilot AI changed the title [WIP] Rename all 'subcommand' references to 'action' Rename subcommand_dictaction_dict and SubcommandProtoActionProto Mar 5, 2026
@hzhangxyz
hzhangxyz marked this pull request as ready for review March 5, 2026 19:08
Copilot AI review requested due to automatic review settings March 5, 2026 19:08
Copilot AI and others added 2 commits March 6, 2026 03:14
Co-authored-by: hzhangxyz <11623447+hzhangxyz@users.noreply.github.com>
@hzhangxyz
hzhangxyz force-pushed the copilot/rename-subcommand-to-action branch from ef73352 to 5e8bf44 Compare March 5, 2026 19:15
@hzhangxyz
hzhangxyz merged commit e70225e into main Mar 5, 2026
8 checks passed
Copilot AI removed the request for review from Copilot March 23, 2026 19:42
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.

所有的 "subcommand"应该替换成 "action"

2 participants