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

Tensor products of commutative algebras do not know they are commutative rings #34453

Closed
tscrim opened this issue Aug 30, 2022 · 11 comments
Closed

Comments

@tscrim
Copy link
Collaborator

tscrim commented Aug 30, 2022

sage: X = algebras.Shuffle(QQ, 'ab')
sage: Y = algebras.Shuffle(QQ, 'bc')
sage: X in CommutativeRings()
True
sage: Y in CommutativeRings()
True
sage: T = tensor([X,Y])
sage: T in CommutativeRings()
False

CC: @mkoeppe @jhpalmieri @fchapoton @mantepse

Component: categories

Author: Travis Scrimshaw

Branch/Commit: 5776c61

Reviewer: Frédéric Chapoton

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

@tscrim tscrim added this to the sage-9.7 milestone Aug 30, 2022
@tscrim tscrim changed the title Tensor products of commutative rings do not know they are commutative Tensor products of commutative algebras do not know they are commutative rings Aug 30, 2022
@tscrim
Copy link
Collaborator Author

tscrim commented Aug 30, 2022

Commit: 5776c61

@tscrim
Copy link
Collaborator Author

tscrim commented Aug 30, 2022

@tscrim
Copy link
Collaborator Author

tscrim commented Aug 30, 2022

New commits:

5776c61Adding the category of commutative rings to the category of tensor products of commutative algebras.

@fchapoton
Copy link
Contributor

comment:3

would this work too for the tensor product of a shuffle algebra A and a polynomial ring B ? maybe add this kind of simple example ?

I tried A.tensor(B) on vanilla sage and it failed.

@tscrim
Copy link
Collaborator Author

tscrim commented Aug 30, 2022

comment:4

Sadly, a polynomial ring does not (yet) know it is an algebra over its base ring.

@fchapoton
Copy link
Contributor

comment:5

the patchbot reports a strange doctest failure, most probably unrelated

if you agree with this diagnostic, you can set to positive

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@tscrim
Copy link
Collaborator Author

tscrim commented Aug 30, 2022

comment:6

I saw that on another ticket, so it is definitely unrelated.

Thank you.

@mantepse
Copy link
Collaborator

comment:7

Replying to Travis Scrimshaw:

Sadly, a polynomial ring does not (yet) know it is an algebra over its base ring.

Could you give an example for this?

sage: P.<x,y> = Integers(8)[]
sage: P in Algebras(P.base_ring())
True

@tscrim
Copy link
Collaborator Author

tscrim commented Sep 16, 2022

comment:8

I must be misremembering then.

@mkoeppe mkoeppe modified the milestones: sage-9.7, sage-9.8 Sep 19, 2022
@vbraun
Copy link
Member

vbraun commented Sep 22, 2022

Changed branch from public/categories/tensor_commutative_rings-34453 to 5776c61

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

5 participants