Skip to content

Conversation

@gift-framework
Copy link
Owner

  • Sort G2 imports alphabetically in PhysLean.lean
  • Add docstrings for notation definitions in Exterior.lean (ε and ∧')
  • Remove unused arguments in E8Lattice.lean:
    • norm_sq_even_of_half_int_even_sum: remove _hsum parameter
    • E8_basis_generates: use term-mode to avoid unused variable
    • reflect_preserves_lattice: remove _hα_root parameter

- Sort G2 imports alphabetically in PhysLean.lean
- Add docstrings for notation definitions in Exterior.lean (ε and ∧')
- Remove unused arguments in E8Lattice.lean:
  - norm_sq_even_of_half_int_even_sum: remove _hsum parameter
  - E8_basis_generates: use term-mode to avoid unused variable
  - reflect_preserves_lattice: remove _hα_root parameter
- Add @[pseudo] to all theorems using native_decide (34 total)
- Fix E8_basis_generates to remove unused membership argument

The @[pseudo] attribute marks theorems that depend on Lean.ofReduceBool
(used by native_decide), as required by PhysLean's sorry_lint checker.
The @[pseudo] attribute is defined in PhysLean.Meta.Linters.Sorry,
not in Mathlib. Add the necessary import to all G2 files that use it.

Also rename unused variable v to _v in E8_basis_generates.
@gift-framework gift-framework merged commit 844642c into master Jan 16, 2026
3 checks passed
@gift-framework gift-framework deleted the claude/physlean-migration-exploration-aRG85 branch January 16, 2026 20: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.

3 participants