-
Notifications
You must be signed in to change notification settings - Fork 55
IsingBimodule for testing multifusion category within TensorKit
#263
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
base: main
Are you sure you want to change the base?
Changes from 12 commits
fb4693f
13bcf82
09467c6
2878e08
10744b2
1ed1ed1
c31b99c
a33d028
0e3c5df
124def4
9635362
4bc1e7c
24be2ea
13af986
095dcda
fbfb564
d7cae4f
a14fc70
4fe82f9
55a00c0
0113f73
8e8b760
78b135e
440b42b
00825af
aad8fc8
c0ef01e
c189284
bdbd2e6
e23e092
06671f3
a18f73d
979036a
279a15f
83d73ce
d7fb9f4
9521574
19ba225
5fef11e
2d9663b
0061588
abb6301
64982d6
520f81d
60bc1fc
490aa4f
8ab417d
6348815
2a7d871
a6bf085
159143e
42fd12b
5393d87
5898bc0
6351cf0
2d107a5
1d3c170
bae646a
99b8d1c
a8e8b81
45ff559
260ef92
cd8e37b
772452f
fd55983
04d6162
5d34b60
d25510a
a3857da
de2bcae
3ea5293
6c20fc6
3dd1257
69e7423
ae10a73
7f01306
8d2e94f
6501e5f
07bcc51
52fa8d4
41013ec
c71b15c
ddc1612
17f1e1b
e8f6a76
df0115e
ac56fae
599cabd
0612656
698d652
9b5ba0b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's not great if we have to specialize everything specifically for There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure what you mean by redoing all that work. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What I mean is that if all we are testing is the specializations that are written solely for IsingBimodule, which is used mainly for testing, this doesn't actually help all that much, since it would have to be both reimplemented and retested for other multifusion categories as well |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What was wrong with the previous implementations? This looks like it will always fail for multifusion things
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted
left/rightoneunitto also work on empty spaces, becauseoneunitdoes so. This will indeed fail for multifusion things, hence the specialisation.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see, but what do you think about this then: