search

Germandrummer92 / TalkingRobot

PSE
0 stars 0 forks source link

RL - "" as recipeName #39

Closed narigaoka closed 10 years ago

narigaoka commented 10 years ago

RL currently does not safe the recipeName anymore (???) we're not sure after which commit this happened, but this causes nullpointerexceptions at various points in our system

narigaoka commented 10 years ago

the problem seems to be related to terms.

i added this in DMPhase: System.out.println(main.getNluResult().get(1)); right before: dialogManager.updateDialog(main.getNluResult().get(0), main.getNluResult().get(1), main.getNluResult().get(3));

and also: for(int i = 0; i < keywords.size(); i++) { System.out.println("working with: " + terms.get(i)); } righ after: public void updateDialog(List keywords, List terms, List approval){ in the dialog manager.

but this is the output:

The robot says: I've found your user history, luiz. How may I help you Your next Input:teach you [teach you, teach you] working with: teach you The robot says: How is this recipe called Your next Input:soup [soup, soup] The robot says: And where does this recipe has it's origins

as you can see the terms for "soup" disappeared ... terms is /null/

alexaand commented 10 years ago

Modifications for better switches between RA and RL were the cause, fixed.