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

Implement the cohomology ring of a right-angled Artin group #29698

Closed
tscrim opened this issue May 17, 2020 · 18 comments
Closed

Implement the cohomology ring of a right-angled Artin group #29698

tscrim opened this issue May 17, 2020 · 18 comments

Comments

@tscrim
Copy link
Collaborator

tscrim commented May 17, 2020

Since they have nice structure corresponding to a quotient of an exterior algebra.

Component: group theory

Keywords: cohomology

Author: Travis Scrimshaw

Branch/Commit: 0c4107c

Reviewer: Frédéric Chapoton

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

@tscrim tscrim added this to the sage-9.1 milestone May 17, 2020
@tscrim
Copy link
Collaborator Author

tscrim commented May 17, 2020

Commit: f33b09b

@tscrim
Copy link
Collaborator Author

tscrim commented May 17, 2020

Branch: public/groups/raag_cohomology-29698

@tscrim
Copy link
Collaborator Author

tscrim commented May 17, 2020

New commits:

f33b09bImplementation of the cohomology ring of a RAAG.

@tscrim tscrim modified the milestones: sage-9.1, sage-9.2 May 17, 2020
@fchapoton
Copy link
Contributor

comment:2

reminds me of #27874

@tscrim
Copy link
Collaborator Author

tscrim commented May 17, 2020

comment:3

It does have the hints of that as the trace monoid is the monoid version of a right-angled Artin group. Although this implementation is quite different IMO.

@fchapoton
Copy link
Contributor

comment:4

some pyflakes warning, see patchbot reports

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 26, 2020

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

abd76f4Merge branch 'public/groups/raag_cohomology-29698' of git://trac.sagemath.org/sage into public/groups/raag_cohomology-29698
966ae57Fixing pyflakes warnings.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 26, 2020

Changed commit from f33b09b to 966ae57

@tscrim
Copy link
Collaborator Author

tscrim commented May 26, 2020

comment:6

Fixed.

@fchapoton
Copy link
Contributor

comment:7

Do you need to use len in these tests ?

+        if len(m) == 0:
+            return '1'
+        term = ''
+        for i in m:
+            if len(term) != 0:
+                term += '*'
+            term += 'e' + str(i)
+        return term

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 28, 2020

Changed commit from 966ae57 to 0c4107c

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 28, 2020

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

0c4107cUsing join and faster bool check in RAAG cohomology.

@tscrim
Copy link
Collaborator Author

tscrim commented May 28, 2020

comment:9

Indeed, I do not. I can also use join() as well.

@tscrim
Copy link
Collaborator Author

tscrim commented Jun 2, 2020

comment:10

Green patchbot.

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:11

Looks good.
I am just so slightly worried about the pdf doc, but was not able to check that it works.
Let me give a positive review nevertheless.

@tscrim
Copy link
Collaborator Author

tscrim commented Jun 3, 2020

comment:12

Replying to @fchapoton:

Looks good.
I am just so slightly worried about the pdf doc, but was not able to check that it works.
Let me give a positive review nevertheless.

Thanks. The pdf doc built for me without any problems.

@vbraun
Copy link
Member

vbraun commented Jun 5, 2020

Changed branch from public/groups/raag_cohomology-29698 to 0c4107c

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