#31941 closed defect (fixed)

Make empty RealSet false

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-9.4
Component: geometry Keywords:
Cc: tscrim Merged in:
Authors: Ralf Stephan Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: 53639be (Commits, GitHub, GitLab) Commit: 53639be9421b66edfe3c1440a511152cbf91d9dd
Dependencies: #31877 Stopgaps:

Status badges

Description (last modified by mkoeppe)

Currently we have:

sage: RealSet()
{}
sage: bool(_)
True

The fix has been split out from the branch of #24176

Change History (4)

comment:1 Changed 11 months ago by mkoeppe

  • Branch set to u/mkoeppe/make_empty_realset_false

comment:2 Changed 11 months ago by mkoeppe

  • Authors set to Ralf Stephan
  • Commit set to 53639be9421b66edfe3c1440a511152cbf91d9dd
  • Description modified (diff)
  • Status changed from new to needs_review

New commits:

d68a69fRealSet.__bool__: New (split out from #24171)
4b09685RealSet: Put it in a suitable subcategory of TopologicalSpaces()
46eed0eRealSet.ambient: Change to a normal method
53639beMerge #31877

comment:3 Changed 11 months ago by tscrim

  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

LGTM.

comment:4 Changed 11 months ago by vbraun

  • Branch changed from u/mkoeppe/make_empty_realset_false to 53639be9421b66edfe3c1440a511152cbf91d9dd
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.