Relax tags parameter type in duplicate_inames to accept string tags

[Copilot]

duplicate_inames required tags: Mapping[str, Tag], rejecting string shorthand values like "unr" that are accepted everywhere else in the API.

Changes

• loopy/transform/iname.py: Change tags parameter type from Mapping[str, Tag] to Mapping[str, ToInameTagConvertible] (str | Tag | None), consistent with how split_iname, chunk_iname, and tag_inames handle tag inputs.

No runtime changes needed — tag_inames (called internally to apply the tags) already handles string→Tag conversion via parse_tag().

Example

# Previously caused a type error; now valid:
lp.duplicate_inames(knl, "i", within=None, tags={"i": "unr"})

# Still works:
lp.duplicate_inames(knl, "i", within=None, tags={"i": UnrollTag()})

---

💡 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]

Pull request overview

This PR aligns duplicate_inames’s tags parameter typing with the rest of Loopy’s iname-tagging API by allowing the same shorthand tag inputs (e.g. "unr") that tag_inames already supports.

Changes:

• Relax duplicate_inames(..., tags=...) type annotation from Mapping[str, Tag] to Mapping[str, ToInameTagConvertible] to accept string tag shorthands (and None).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.