facebook / duckling

Language, engine, and tooling for expressing, testing, and evaluating composable language rules on input strings.
Other
4.08k stars 726 forks source link

[AF][Numeral] CI test failed on "agt" #572

Open chessai opened 3 years ago

chessai commented 3 years ago

"agt" means 8 (AF). There is seemingly no reason for this test to fail out of nowhere, especially with no change to AF code. In the meantime I have restarted the GitHub CI jobs for this. This issue is to track progress on this in case it persists. See:

> debug (makeLocale AF Nothing) "agt" [Seal Numeral]
number (0..10) (agt)
-- regex (agt)
[Entity {dim = "number", body = "agt", value = RVal Numeral (NumeralValue {vValue = 8.0}), start = 0, end = 3, latent = False, enode = Node {nodeRange = Range 0 3, token = Token Numeral (NumeralData {value = 8.0, grain = Nothing, multipliable = False, okForAnyTime = True}), children = [Node {nodeRange = Range 0 3, token = Token RegexMatch (GroupMatch ["agt"]), children = [], rule = Nothing}], rule = Just "number (0..10)"}}]
> :test Duckling.Numeral.AF.Tests
AF Tests
  Corpus Tests: OK

All 1 tests passed (0.02s)
chessai commented 3 years ago

Stack builds don't fail, but cabal builds do. I wonder if there is some weird dependency discrepancy causing a bug.

chessai commented 3 years ago

Switching to stack solved the problem. I don't know why.

chessai commented 3 years ago

See https://github.com/facebook/duckling/pull/605#issuecomment-822072200