Skip to content

Prefix automaton -- julia bug?#75

Open
kalmarek wants to merge 13 commits intomasterfrom
mk/prefix_automaton
Open

Prefix automaton -- julia bug?#75
kalmarek wants to merge 13 commits intomasterfrom
mk/prefix_automaton

Conversation

@kalmarek
Copy link
Copy Markdown
Owner

No description provided.

@kalmarek kalmarek force-pushed the mk/prefix_automaton branch from 76eb55c to a84e1dd Compare July 31, 2024 15:07
@kalmarek kalmarek force-pushed the mk/prefix_automaton branch from a84e1dd to 64691e6 Compare July 31, 2024 15:11
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 31, 2024

Codecov Report

Attention: Patch coverage is 70.29703% with 60 lines in your changes missing coverage. Please review.

Project coverage is 92.41%. Comparing base (690cfa8) to head (64691e6).

Files Patch % Lines
src/Automata/prefix_automaton.jl 43.29% 55 Missing ⚠️
src/Automata/index_automaton.jl 96.00% 1 Missing ⚠️
src/Automata/rebuilding_idxA.jl 50.00% 1 Missing ⚠️
src/knuthbendix_idxA.jl 88.88% 1 Missing ⚠️
src/rewriting.jl 97.43% 1 Missing ⚠️
src/rules.jl 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
- Coverage   94.90%   92.41%   -2.49%     
==========================================
  Files          30       30              
  Lines        1883     1992     +109     
==========================================
+ Hits         1787     1841      +54     
- Misses         96      151      +55     
Flag Coverage Δ
unittests 92.41% <70.29%> (-2.49%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kalmarek kalmarek changed the title Prefix automaton Prefix automaton -- julia bug? Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants