Skip to content

Remove 20 ForMathlib declarations now in mathlib/core; rename lsbs → setWidth - #3

Open
ibrahimmian36 wants to merge 1 commit into
inQWIRE:mainfrom
ibrahimmian36:triage/delete-redundant
Open

ibrahimmian36 wants to merge 1 commit into
inQWIRE:mainfrom
ibrahimmian36:triage/delete-redundant

Conversation

@ibrahimmian36

Copy link
Copy Markdown

These are all provided by mathlib or Lean core at the pinned toolchain (v4.30.0-rc2, mathlib c1e30e17). Builds green.

Proofs re-deriving each deleted statement from mathlib alone, plus a triage of the whole ForMathlib/ directory: https://github.com/ibrahimmian36/leanquantum-triage

One proof needed a small change: weight_and_le relied on lsbs being opaque to simp; the setWidth version uses the IH and omega.

…me lsbs->setWidth at use sites

Verified against mathlib c1e30e17 (the lake-manifest revision) on
leanprover/lean4:v4.30.0-rc2. One proof adjustment disclosed:
weight_and_le's original proof relied on lsbs being opaque to simp;
the setWidth version instantiates the induction hypothesis and closes
with omega. Evidence that every deleted statement is derivable from
mathlib/core alone: leanquantum-triage/evidence/Redundant.lean.
@rnrand

rnrand commented Sep 8, 2026

Copy link
Copy Markdown
Member

This looks good to me.

@f64u, okay to merge?

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.

2 participants