[FEAT] Drop Padatious for Padacioso
NeonDaniel opened this issue · 0 comments
NeonDaniel commented
Objective
Padatious is inconsistent with intent matches, and often matches to liberally. Padacioso has been implemented in ovos-core for at least 2 stable releases and has been performing better in unit tests.
Initial Implementation Requirements
- Implement default Padacioso intent parser in NeonCore
- Consider removing Padatious requirements
Other Considerations
https://github.com/NeonGeckoCom/skill-date_time/actions/runs/4756660605/jobs/8452511151?pr=56
Unit tests should have some threshold for passing padatious (maybe skipping negative tests?)