Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding essential supremum for extended reals #1361

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hoheinzollern
Copy link
Collaborator

Co-authored-by: @jmmarulang

Motivation for this change

Addresses issue #1360

Checklist
  • added corresponding entries in CHANGELOG_UNRELEASED.md

  • added corresponding documentation in the headers

Reference: How to document

Reminder to reviewers

Added documentation and changelog

Co-authored-by: @jmmarulang
Copy link
Contributor

@zstone1 zstone1 left a comment

Choose a reason for hiding this comment

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

The definition and namings make sense. One optional generalization that might be nice, if it's easy. But it looks good to me.

Definition ess_supe f :=
ereal_inf ([set r | mu (f @^-1` `]r, +oo[) = 0]).

Lemma ess_supe_ge0 f : 0 < mu [set: T] -> (forall t, 0 <= f t) ->
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like this should generalize easily to any a which is a lower bound on x. That is, ess_supe_ge0 a f : 0 < mu [set: T] -> (forall t, a <= f t) -> a <= ess_supe f. If it's easy to do that, we should. But definitely not a blocker.

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