microsoft / botframework-components

The repository for components built by Microsoft for the Azure Bot Framework.
https://aka.ms/botdocs
MIT License
113 stars 82 forks source link

Error in adding PointOfInterest skill to VA. #279

Closed VatsalaSingh closed 4 years ago

VatsalaSingh commented 4 years ago

I'm trying to add PointOfInterest skill to my Virtual Assistant. I'm getting the following error. In this snapshot, I'm currently in my VA directory. I've added the LUIS path of the skill. skillerror

How can I resolve this?

xieofxie commented 4 years ago

Hi @VatsalaSingh I have tested it on master of this repo and solutions repo and it worked correctly. Do you use the latest tool? https://www.npmjs.com/package/botskills

VatsalaSingh commented 4 years ago

@xieofxie yes I'm using the latest tool. First I created a skill project from VS (https://microsoft.github.io/botframework-solutions/skills/tutorials/create-skill/csharp/3-create-your-skill/) and added that to my VA. That successfully worked. The issue seems to be with sample skills.

xieofxie commented 4 years ago

@VatsalaSingh Oh, I figured out the reason from your screen shot. It seems the tool forget to quote bf luis:convert for the out parameter. Note the Command is bf luis:convert ... --out C:\Users\vasingh\Desktop\sample skills\botframework-skills-master ... and the error is unexpected argument skills\botframework-skills-master ....

If you put it in a path without space it should be fine and I will create an issue about this.

VatsalaSingh commented 4 years ago

Right, it works now. Thanks :)