#31243 closed enhancement (fixed)

Stronger Categories for Manifold Examples: Connectedness

Reported by: Michael Jung Owned by:
Priority: major Milestone: sage-9.3
Component: manifolds Keywords:
Cc: Eric Gourgoulhon, Travis Scrimshaw Merged in:
Authors: Michael Jung Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: 440bc50 (Commits, GitHub, GitLab) Commit: 440bc5053d601a29538ac1640729f3719abe8932
Dependencies: #31233 Stopgaps:

Status badges

Description (last modified by Michael Jung)

Currently, an instance of OpenInterval just belongs to the category of smooth manifolds even though it is also connected. Similarly for Euclidean spaces and spheres.

The category of connected manifolds will be joined.

Part of metaticket #30189.

Change History (16)

comment:1 Changed 23 months ago by Michael Jung

Description: modified (diff)

comment:2 Changed 23 months ago by Michael Jung

Branch: u/gh-mjungmath/stronger_categories_manifolds_examples

comment:3 Changed 23 months ago by Michael Jung

Commit: 81a94a550ce71b638af365b3f7f2e82708c77234
Status: newneeds_review

New commits:

81a94a5Trac #31243: examples added to category of connected manifolds

comment:4 Changed 23 months ago by git

Commit: 81a94a550ce71b638af365b3f7f2e82708c77234c3c7ff0b05eb1d36c011bebfd4579fdb99a9a938

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

c3c7ff0Trac #31243: examples added to category of connected manifolds

comment:5 Changed 23 months ago by Michael Jung

Dependencies: #31233

comment:6 Changed 23 months ago by git

Commit: c3c7ff0b05eb1d36c011bebfd4579fdb99a9a9381483634cf5d6084698482ec417dbcb8e57358b6c

Branch pushed to git repo; I updated commit sha1. New commits:

be52d78Trac #31233: meet categories
78f7af3Trac #31233: category of homset docstring adapted
8d31e0fTrac #31233: fixing format
111015dTrac #31233: adapt homsets in docstring
68de9c9Merge branch 't/31233/categories_for_homset_of_manifolds_do_not_meet' into stronger_categories_manifolds_examples
1483634Trac #31243: doctest conflicts with #31233

comment:7 Changed 23 months ago by Travis Scrimshaw

It would be easier just to call the Connected() on the category:

-        category = Manifolds(RR).Smooth() & Manifolds(RR).Connected()
+        category = Manifolds(RR).Smooth().Connected()
-                       TopologicalSpaces().Compact() & Manifolds(RR).Connected()
+                       TopologicalSpaces().Compact().Connected()
Last edited 23 months ago by Travis Scrimshaw (previous) (diff)

comment:8 Changed 23 months ago by git

Commit: 1483634cf5d6084698482ec417dbcb8e57358b6c7f0de49bcd13308928238a598122b0995749fbee

Branch pushed to git repo; I updated commit sha1. New commits:

7f0de49Trac #31243: doctest missed

comment:9 in reply to:  7 Changed 23 months ago by Michael Jung

Replying to tscrim:

It would be easier just to call the Connected() on the category:

-        category = Manifolds(RR).Smooth() & Manifolds(RR).Connected()
+        category = Manifolds(RR).Smooth().Connected()
-                       TopologicalSpaces().Compact() & Manifolds(RR).Connected()
+                       TopologicalSpaces().Compact().Connected()

Indeed.

comment:10 Changed 23 months ago by git

Commit: 7f0de49bcd13308928238a598122b0995749fbee22df7718f71eb2a323bdf7e4838635c87b0b7edd

Branch pushed to git repo; I updated commit sha1. New commits:

22df771Trac #31241: call categories more comfortable + rearranged category order in doctest

comment:11 Changed 23 months ago by Travis Scrimshaw

Thank you. Green bot => positive review.

comment:12 Changed 23 months ago by git

Commit: 22df7718f71eb2a323bdf7e4838635c87b0b7edd440bc5053d601a29538ac1640729f3719abe8932

Branch pushed to git repo; I updated commit sha1. New commits:

440bc50Trac #31243: categories adapted in tutorial

comment:13 Changed 23 months ago by Michael Jung

Patchbot green.

comment:14 Changed 23 months ago by Travis Scrimshaw

Reviewers: Travis Scrimshaw
Status: needs_reviewpositive_review

comment:15 Changed 23 months ago by Michael Jung

Thanks.

comment:16 Changed 23 months ago by Volker Braun

Branch: u/gh-mjungmath/stronger_categories_manifolds_examples440bc5053d601a29538ac1640729f3719abe8932
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.