leanprover-community / leanprover-community/mathlib4

Lightweight variant of `group`

Open
#21,324 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Lean
Stars
4.2k
Forks
1.7k
PR merge metrics
No merged PRs in 30d

Description

group is a very heavy tactic (it imports ring to normalise exponents) which is no more powerful than simp [mul_assoc] in many cases appearing in practice. It might be useful to have a lightweight variant of group (in the sense that it doesn't import Field or even MonoidWithZero) for those numerous cases.

Zulip

List of PRs removing uses of group:

  • #21296

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

Start by reading the group tactic and the import concerns described in the issue, especially the roles of ring, Field, and MonoidWithZero. Review PR #21296 and the linked Zulip discussion to understand existing removals and the intended lightweight scope; done means a suitable variant is defined and the relevant heavy imports are no longer required for those cases.

Written by the indexing model from the issue text.

Assessment

Domain
tooling
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.