Skip to content

fix: Exercise 1.1.7(i) needs a closed box - #648

Open
Chessing234 wants to merge 1 commit into
teorth:mainfrom
Chessing234:fix/exercise-1-1-7-closed-box
Open

fix: Exercise 1.1.7(i) needs a closed box#648
Chessing234 wants to merge 1 commit into
teorth:mainfrom
Chessing234:fix/exercise-1-1-7-closed-box

Conversation

@Chessing234

Copy link
Copy Markdown
Contributor

A Box d is a product of BoundedIntervals, which need not be closed, and ContinuousOn f B.toSet over a non-closed box does not force f to be bounded.

Counterexample to the statement as formalized: take d = 1, B.side 0 = Ioo 0 1, and f x = 1 / x 0 (away from 0, so ContinuousOn f B.toSet holds). Then

{ p | ∃ x ∈ B.toSet, prod_equiv 1 1 p = ⟨x, f x⟩ }

is unbounded in its second coordinate, and so is the region under it. JordanMeasurable carries a boundedness field, so neither set is Jordan measurable and both graph and undergraph are false.

Adding (hB : IsClosed B.toSet) makes B.toSet compact, so a continuous f is bounded and uniformly continuous there — the closed-box hypothesis the textbook exercise uses. IsClosed B.toSet is how closed boxes are already spelled in Section_1_2_1.lean.

measure_of_graph picks up the hypothesis too since it refers to JordanMeasurable.graph. All three bodies stay sorry, and none of the three lemmas is referenced elsewhere in the repository.

A Box is a product of BoundedIntervals, which need not be closed, and
ContinuousOn f B.toSet on a non-closed box does not force f to be
bounded.  Taking d = 1, B = Ioo 0 1 and f x = 1 / x (continuous on
B.toSet) makes the graph, and the region under it, unbounded, so neither
is Jordan measurable -- JordanMeasurable carries a boundedness field.

Requiring IsClosed B.toSet makes B.toSet compact, so f is bounded and
uniformly continuous there, which is the hypothesis the textbook
exercise uses.  IsClosed B.toSet is the same way closed boxes are
spelled elsewhere (Section_1_2_1).

Signed-off-by: Taksh <takshkothari09@gmail.com>
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.

1 participant