Skip to content

feat/disambiguation#18

Merged
JarbasAl merged 2 commits intodevfrom
feat/disambiguation
Jul 12, 2023
Merged

feat/disambiguation#18
JarbasAl merged 2 commits intodevfrom
feat/disambiguation

Conversation

@JarbasAl
Copy link
Member

with previous PR we lost determinism due to parallelization, that highlighted disambiguation issues and made unittests randomly pass or fail

better disambiguation during fuzzy intents based on number of entities and len() difference between query and samples

makes things slightly more deterministic

with previous PR we lost determinism due to parallelization, that highlighted disambiguation issues and made unittests randomly pass or fail

better disambiguation during fuzzy intents based on number of entities and len() difference between query and samples

makes things slightly more deterministic
@JarbasAl JarbasAl added bug Something isn't working enhancement New feature or request labels Jul 12, 2023
@JarbasAl JarbasAl requested a review from NeonDaniel July 12, 2023 13:06
@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Merging #18 (dae015e) into dev (069fe2e) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff          @@
##             dev     #18   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          3       3           
  Lines        247     270   +23     
=====================================
- Misses       247     270   +23     
Impacted Files Coverage Δ
padacioso/__init__.py 0.00% <0.00%> (ø)
padacioso/version.py 0.00% <0.00%> (ø)

@JarbasAl JarbasAl merged commit 0b74a74 into dev Jul 12, 2023
@JarbasAl JarbasAl deleted the feat/disambiguation branch July 12, 2023 13:09
@github-actions github-actions bot mentioned this pull request Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant