Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Js/nc groebner optimization stable #1292
Js/nc groebner optimization stable #1292
Changes from all commits
24cb8c2
d80fd56
ebf71e3
2e851e1
8210424
f337310
d8fa4d8
adfad26
910674b
8de2e0a
46c8f38
e51bfa7
c665b34
a53d92e
3a5cfde
022819c
0933b0f
135c935
0f7fe94
1549e89
20db65b
51da2f6
21da71b
24adfb8
9c6d94c
ff19858
fb393b2
2abd28f
3586f2f
e41b0b9
d6cb270
6602fc7
6f9afbb
d1e4787
575ea86
88c6df6
2106c6c
b372561
17eb859
8c3fcc5
804c4bf
d04a58e
b232a4e
6bda614
6fca238
8582322
9c75436
12ee122
4df7adf
f801367
68842fa
70601eb
20f9b7b
dfe9371
817d467
a964c18
8e3cae3
e9d34e7
d74b254
6739af9
344f3c6
df7c3e1
6393fc5
c0e094d
5e3880f
9662933
e595189
10b9134
6483879
4b2b049
0482a64
4630e05
3e8d387
f5f165d
b00add0
8ec6ef3
f06043a
955f980
8f90c18
e5d32be
9a29b09
1bb6853
a3dd379
29d7b37
56c1305
e433bd3
eb666e9
64a8871
5e55c87
9cce960
62185df
80578a9
8f223b5
9449acb
9a37fd1
03527de
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Check warning on line 127 in src/generic/AhoCorasick.jl
Codecov / codecov/patch
src/generic/AhoCorasick.jl#L126-L127
Check warning on line 159 in src/generic/AhoCorasick.jl
Codecov / codecov/patch
src/generic/AhoCorasick.jl#L158-L159
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.
So you are essentially reconstructing the whole failure function from scratch.
You may have already found this (it's linked on Wikipedia), but just in case not: it seems that there is an incremental version of Aho-Corasick which might be interesting for you (I have no idea if
insert_keyword!
ever is a bottleneck for you, though). Anyway, see https://se.inf.ethz.ch/~meyer/publications/string/string_matching.pdf -- I did not check this in detail, but the relevant section doesn't seem very long. If this function ever shows up in your profiling, it might be worth experimenting with it?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.
Thanks for the suggestion, that looks nice, however in my computations
insert_keyword!
is mostly irrelevant in terms of running time. The vast majority of time is spent innormal_form
, roughly 95% in some examples and I think the relevance ofnormal_form
increases with the number of relations.Check warning on line 425 in src/generic/FreeAssAlgebra.jl
Codecov / codecov/patch
src/generic/FreeAssAlgebra.jl#L425