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

Mixed form name bug for pullback bundle #31784

Closed
mjungmath opened this issue May 6, 2021 · 15 comments
Closed

Mixed form name bug for pullback bundle #31784

mjungmath opened this issue May 6, 2021 · 15 comments

Comments

@mjungmath
Copy link

sage: E3 = EuclideanSpace(3)
sage: S2 = E3.sphere()
sage: iota = S2.embedding()
sage: Omega = S2.mixed_form_algebra(dest_map=iota)
sage: Omega(1)
Mixed differential form one along the {} with values on the {} Mixed differential form one along the 2-sphere S^2 of radius 1 smoothly embedded in the Euclidean space E^3 with values on the Euclidean space E^3 via the map iota

CC: @egourgoulhon @tscrim

Component: manifolds

Author: Michael Jung

Branch/Commit: bd5031b

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/31784

@mjungmath mjungmath added this to the sage-9.4 milestone May 6, 2021
@mjungmath
Copy link
Author

Author: Michael Jung

@mjungmath

This comment has been minimized.

@mjungmath
Copy link
Author

@mjungmath
Copy link
Author

New commits:

af19b9cTrac 31784: fix repr for destination maps

@mjungmath
Copy link
Author

Commit: af19b9c

@mjungmath
Copy link
Author

comment:4

We need more examples for those cases. At the moment, this is not caught by many tests. Similarly for tensor fields by the way.

@mjungmath
Copy link
Author

comment:5

But this should do for the moment.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 6, 2021

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

bd5031bTrac 31784: fix repr for destination maps

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 6, 2021

Changed commit from af19b9c to bd5031b

@mjungmath

This comment has been minimized.

@mjungmath
Copy link
Author

comment:8

Patchbot green.

@tscrim
Copy link
Collaborator

tscrim commented May 7, 2021

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented May 7, 2021

comment:9

LGTM.

@mjungmath
Copy link
Author

comment:10

Thank you for the review.

@vbraun
Copy link
Member

vbraun commented Jun 19, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants