Skip to content
Snippets Groups Projects

Add lower bound lemma for finite sets

Merged Lennard Gäher requested to merge ci/lennard/lowerbound into master
All threads resolved!

Forked off from the discussion here: lambda-rust!38 (comment 104108)

This adds a variant of minimal_exists which does not require the set to be non-empty, but in return also does not guarantee that the element is part of the set.

I'm not sure what the best naming for this lemma is. Maybe it should be called infimum_exists instead of lower_bound_exists? Should it be prefixed by set, like many other lemmas in the file for which it is not directly clear it's related to sets? (and should minimal_exists be prefixed accordingly?)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Thanks.

    I'm not sure what the best naming for this lemma is. Maybe it should be called infimum_exists instead of lower_bound_exists?

    No strong opinion.

    Should it be prefixed by set, like many other lemmas in the file for which it is not directly clear it's related to sets? (and should minimal_exists be prefixed accordingly?)

    The convention is a bit messy, so it's not clear. I wouldn't change the other lemma now, since the definition minimal already does not have a set_ prefix. The proposed name does not involve minimal, so it's less clear whether to include set_ or not. I don't have a strong opinion.

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Lennard Gäher added 3 commits

    added 3 commits

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Lennard Gäher resolved all threads

    resolved all threads

  • Ralf Jung
  • added 1 commit

    Compare with previous version

  • Lennard Gäher resolved all threads

    resolved all threads

  • LGTM, thanks! I'll take care of updating dependencies that use the old lemma name.

  • Ralf Jung mentioned in commit 8548ce42

    mentioned in commit 8548ce42

  • merged

  • @iris-users this renames a lemma, you may have to run a sed script to update your proofs:

    • Rename minimal_exists to minimal_exists_elem_of and minimal_exists_L to minimal_exists_elem_of_L.
  • Please register or sign in to reply
    Loading