search

informatics-lab / weather-bot

Met Office prototype chat bot to talk about all things weather & climate related
1 stars 1 forks source link

Doesn't handle dates in the dd/mm/yy format well #49

Closed tam203 closed 7 years ago

tam203 commented 7 years ago

image

Don't know if this is Sol or LUIS issue.

tam203 commented 7 years ago

Turns out to be annoying. Its easy to start recognising dd/mm/yy just put some examples in the LUIS utterances. LUIS datetimeV2 will automatically pick it out as a date. However, it will assume US style mm/dd/yy which will then break the app in many places.

We would have to add a range of parsing in the capture stage to handle this. Look at the actual text sent for the entity and re-parse if we think it looks like a date that could of been misinterpreted.

tpowellmeto commented 7 years ago

Possibly raise this as a bug/improvement on the botbuilder/luis projects - ability to set some kind of locale for parsing data from text. Currently we have no control over this so I'm closing the issue.

tam203 commented 7 years ago

Have raised - https://github.com/Microsoft/Cognitive-LUIS-Windows/issues/55 . We could do an internal fix perhaps but not worth the effort and likely fragile.