Skip to content

chore(RepresentationTheory/Coinduced): more tech debt - #43959

Open
JX-Mo wants to merge 4 commits into
leanprover-community:masterfrom
JX-Mo:resCoindTechDebt
Open

JX-Mo wants to merge 4 commits into
leanprover-community:masterfrom
JX-Mo:resCoindTechDebt

Conversation

@JX-Mo

@JX-Mo JX-Mo commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added WIP Work in progress tech debt Tracking cross-cutting technical debt, see e.g. the "Technical debt counters" stream on zulip labels Sep 19, 2026
@github-actions

github-actions Bot commented Sep 19, 2026

Copy link
Copy Markdown

PR summary 880209271a

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff (regex)

+ coind_apply_apply
+ resCoindAdjunction_counit_app_hom_toFun
+ resCoindAdjunction_homEquiv

You can run this locally as follows
## from your `mathlib4` directory:
git clone https://github.com/leanprover-community/mathlib-ci.git ../mathlib-ci

## summary with just the declaration names:
../mathlib-ci/scripts/pr_summary/declarations_diff.sh <optional_commit>

## more verbose report:
../mathlib-ci/scripts/pr_summary/declarations_diff.sh long <optional_commit>

The doc-module for scripts/pr_summary/declarations_diff.sh in the mathlib-ci repository contains some details about this script.

Declarations diff (Lean)

Lean-aware diff — post-build, computed from the Lean environment (commit 8802092).

  • +4 new declarations
  • −0 removed declarations
+Rep.resCoindAdjunction_counit_app_hom_toFun
+Rep.resCoindAdjunction_homEquiv
+Rep.resCoindAdjunction_unit_app_hom_toFun_coe
+Representation.coind_apply_apply

Decrease in strong tech debt: (relative, absolute) = (1.57, 0.00)
Current number Change Type (strong)
backward.isDefEq.respectTransparency.types 2356 -5
adaptation notes 394 -1
No changes to weak technical debt.

Current commit 880209271a
Reference commit f0ab343610

This script lives in the mathlib-ci repository. To run it locally, from your mathlib4 directory:

git clone https://github.com/leanprover-community/mathlib-ci.git ../mathlib-ci
../mathlib-ci/scripts/reporting/technical-debt-metrics.py pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@github-actions github-actions Bot added the t-algebra Algebra (groups, rings, fields, etc) label Sep 19, 2026
@JX-Mo

JX-Mo commented Sep 19, 2026

Copy link
Copy Markdown
Contributor Author

!radar

@leanprover-radar

leanprover-radar commented Sep 19, 2026

Copy link
Copy Markdown

Benchmark results for 9725ea0 against f0ab343 are in. No significant results found. @JX-Mo

  • build//instructions: -8.4G (-0.01%)

Small changes (5✅)

  • build/module/Mathlib.MeasureTheory.Measure.CharacteristicFunction//instructions: -76.9M (-1.65%)
  • build/module/Mathlib.RepresentationTheory.Coinduced//instructions: -3.3G (-8.32%) (reduced significance based on *//lines)
  • build/module/Mathlib.RepresentationTheory.Homological.GroupCohomology.Shapiro//instructions: -712.6M (-5.51%)
  • build/module/Mathlib.Tactic.Convert//instructions: -92.1M (-0.98%)
  • build/module/Mathlib.Tactic.GRewrite//instructions: -23.6M (-1.19%)

@JX-Mo JX-Mo changed the title chore(RepresentationTheory/Coinduced): no_index tech debt chore(RepresentationTheory/Coinduced): more tech debt Sep 19, 2026
@github-actions github-actions Bot removed the WIP Work in progress label Sep 19, 2026
even with all inputs explicitly given, the first universe cannot be synthesized.
-/
@[simps, pp_with_univ]
@[simps -isSimp, pp_with_univ]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not completely sure about this -isSimp. It is here because it fires unnecessarily in FiniteIndex.lean

Comment on lines +272 to +273
map_add' _ _ := by apply Rep.hom_ext; ext; rfl
map_smul' _ _ := by apply Rep.hom_ext; ext; rfl

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The original raw rlfs are expensive

@JX-Mo JX-Mo Sep 19, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ext does not fire directly, so I have to writeapply Rep.hom_ext (which is tagged @[ext] however). I didn't figure this out, but it works anyway

@JX-Mo

JX-Mo commented Sep 19, 2026

Copy link
Copy Markdown
Contributor Author

WIP

@github-actions github-actions Bot added the WIP Work in progress label Sep 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-algebra Algebra (groups, rings, fields, etc) tech debt Tracking cross-cutting technical debt, see e.g. the "Technical debt counters" stream on zulip WIP Work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants