leanprover-community / leanprover-community/physlib

Documentation claims anomaly cancellation is preserved under mapping without

Open Beginner friendly
#1,541 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

good first issue help-wanted
Dominant language
Lean
Stars
749
Forks
189
Avg merge
1d 21h
Merged PRs (30d)
75

Description

⚠️ Found by a local LLM, not a human. This was flagged by an automated documentation-mistake finder (qwen3:14b) and passed 10/10 independent verification runs before filing, but it has not been reviewed by a person. Please check it's a genuine error before acting on it.


Summary

The documentation in Physlib/Particles/SuperSymmetry/SU5/ChargeSpectrum/Map.lean (around line 21) is incorrect.

Current text
There are various properties which are preserved under this mapping:
- Anomaly cancellation.
- The presence of a specific term in the potential.
- Being complete.
Why this is wrong

The documentation in physlib/Physlib/Particles/SuperSymmetry/SU5/ChargeSpectrum/Map.lean incorrectly states that "anomaly cancellation" is preserved under the map function, but no lemma in the file supports this assertion. The module defines properties like map_allowsTerm and map_isComplete_iff, which correspond to the documented preservation of potential terms and completeness, respectively. However, there is no declaration or proof related to anomaly cancellation in the code.

The correction removes "anomaly cancellation" from the list of preserved properties, aligning the documentation with the actual implementation. This change ensures that only properties with corresponding lemmas (like map_allowsTerm and map_isComplete_iff) are claimed to be preserved under the mapping. The remaining claims about potential terms and completeness are supported by explicit proofs in the code.

Suggested correction
There are various properties which are preserved under this mapping:
- The presence of a specific term in the potential.
- Being complete.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Open Physlib/Particles/SuperSymmetry/SU5/ChargeSpectrum/Map.lean around lines 21–24 and compare the documented properties with declarations such as map_allowsTerm and map_isComplete_iff. Confirm that no anomaly-cancellation preservation result is present, then update the list so it only claims the supported properties; done means the documentation matches the implementation.

Written by the indexing model from the issue text.

Assessment

Domain
documentation
Issue type
Documentation
Difficulty
1/5
Estimated time
Under an hour
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
85/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.