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 dual equivalence graphs #18050
Comments
Commit: |
New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:4
some failing doctests, see patchbot report |
comment:5
In other news, here's something I've been long since curious about:
What is the difference between the syntax with the tilde and that without? |
comment:6
The tilde only displays the last thing after a dot, without displays the entire path. |
comment:7
Ah, thank you! |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:10
Stupid question: Assuming that your dual equivalence graph is the graph |
comment:11
The |
comment:12
I think the docstring of
|
comment:13
Replying to @tscrim:
Oh. Is |
comment:14
Replying to @darijgr:
Yes.
It means that we can do the @mantepse I will make those changes in a minute. |
Branch pushed to git repo; I updated commit sha1. New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:34
Positive review by Travis who was standing over my shoulders and explaining what I should not do. :P |
Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:
|
comment:36
Couldn't resist fixing a few docstrings which made an iterator sound like it was a list and lists pretend to be iterators. Can you review this last bit, Travis? Thanks! |
comment:37
NP. |
comment:38
For your coloring scheme, why don't you do it with a dictionary rather than the long list of if statements? Anne |
Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:41
That's a much better method to get the colors. I also pulled in #18032 because we needed to get immutable copies of the graph so I could write a custom cache which does not regenerate the graph when we only want a different coloring. I also marked a test as long since it took nearly 20s on my computer. Someone check my changes please. |
Dependencies: #18032 |
Branch pushed to git repo; I updated commit sha1. New commits:
|
Changed branch from public/crystals/dual_equivalence-18050 to |
Implement methods for dual equivalence graphs coming from standard tableaux of a fixed shape and as weight 0 crystal spaces.
Depends on #18032
CC: @sagetrac-sage-combinat @anneschilling @darijgr @sagetrac-troby
Component: combinatorics
Keywords: dual equivalence graphs, sd67
Author: Travis Scrimshaw
Branch/Commit:
7594b95
Reviewer: Darij Grinberg
Issue created by migration from https://trac.sagemath.org/ticket/18050
The text was updated successfully, but these errors were encountered: