Skip to content

Comments

Fix abs_finite_series_le signature and add Exercise 7.1.6/7.1.7 state…#448

Merged
teorth merged 1 commit intoteorth:mainfrom
rkirov:upstream-statement-fixes
Feb 19, 2026
Merged

Fix abs_finite_series_le signature and add Exercise 7.1.6/7.1.7 state…#448
teorth merged 1 commit intoteorth:mainfrom
rkirov:upstream-statement-fixes

Conversation

@rkirov
Copy link
Contributor

@rkirov rkirov commented Feb 15, 2026

…ments

  • Remove unused parameter (c:ℝ) from abs_finite_series_le and its call site
  • Fix extra whitespace in finite_series_const_mul signature
  • Add Exercise 7.1.6 (sum over disjoint partition) statement
  • Add Exercise 7.1.7 (column-row double counting) statement

theorem sum_union_disjoint {X S : Type*} [Fintype X] [Fintype S]
(E : X → Finset S)
(disj : ∀ i j : X, i ≠ j → Disjoint (E i) (E j))
(cover : ∀ s : S, ∃ i, s ∈ E i)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There were a few different ways to write this in type theory, e.g. using explicit call to Finset.biUnion. Let me know if another one is preferred. I thought this one is the cleanest.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I ended up writing a whole blog post based on this https://rkirov.github.io/posts/sets-vs-types/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the mathlib version is here https://leanprover-community.github.io/mathlib4_docs/Mathlib/Algebra/BigOperators/Group/Finset/Basic.html#Finset.sum_biUnion but it strikes me as unnecessarily general for the purposes of this book.

Copy link
Owner

Choose a reason for hiding this comment

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

Looks good to me.

…ments

- Remove unused (c:ℝ) parameter from abs_finite_series_le
- Fix whitespace in finite_series_const_mul and abs_finite_series_le
- Add Exercise 7.1.6 (sum over disjoint partition) with Fin n index
- Add Exercise 7.1.7 (column-row counting identity) with Fin types

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rkirov rkirov force-pushed the upstream-statement-fixes branch from 42b00fb to 698277c Compare February 15, 2026 23:39
@teorth teorth merged commit 4ff55db into teorth:main Feb 19, 2026
3 checks passed
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