microsoft / botframework-cli

CLI Tools for the Microsoft Bot Framework
Other
176 stars 129 forks source link

luis:convert "culture was unexpected at this time" #450

Closed rvallireply closed 4 years ago

rvallireply commented 4 years ago

Hello trying to convert the command in the samples' luis_function.ps1

    ludown parse toluis `
        --in $lu_file `
        --luis_culture $language `
        --out_folder $outFolder `
        --out $outFile  

to the CLI command

bf luis:convert --in="($lu_file)" --culture="($language)" --out="$(Join-Path $outFolder $outFile)"

I am getting this

--culture was unexpected at this time.

Is it an error or warning? it seems the language is correctly recognized anyway and the correct LUIS app culture created.

BTW: Any idea if/when the deployment scripts in the samples will be updated with the CLI version commands?

a-b-r-o-w-n commented 4 years ago

Hi @rvallireply what luis_function.ps1 script are you referring to? Can you please provide a link to it?

rvallireply commented 4 years ago

Hi,

Here is the link to the sample script folder, it's in the ITSM skill sample

https://github.com/microsoft/botframework-solutions/tree/master/skills/csharp/experimental/itsmskill/Deployment/Scripts

munozemilio commented 4 years ago

Hello @rvallireply, does the CLI command fails if you run it directly in the terminal? @lauren-mills Do we have a target for the scripts migration?

lauren-mills commented 4 years ago

@rvallireply - The 0.8 release in the botframework-solutions repo yesterday includes the migration to bf cli for the ludown, luisgen, and qnamaker. bf luis support will be added in a subsequent release. Please refer to the latest ITSM skill sample in master for the updates.

munozemilio commented 4 years ago

Hello @rvallireply, Im closing this since the scripts are released now. Feel free to open again if you face any other issue