MicrosoftLearning / AI-102-AIEngineer

Lab files for AI-102 - AI Engineer
https://microsoftlearning.github.io/AI-102-AIEngineer/
MIT License
735 stars 840 forks source link

Lab 11 Using Wav file not in LODS #59

Closed AndrewConniff closed 3 years ago

AndrewConniff commented 3 years ago

Module: 05

Lab/Demo: 11 PYTHON

Task: 1

Step: 06

Description of issue If the user cannot use the microphone there is a wave file to use. The wave file provided is time-in-london.wav the updated instructions did not appear in the lab - also ir referenced a time.wav which was not added into the lab folder, and also the name of the the recognizer is listed as speech_recognizer but in the code it is 'recognizer'.

I was able to get it to work for the user by fixing the code but the wave file asked questions that the LUIS and code was not set up to handle - in short this was a mess - but may be from it not publishing to LODS and having to work with the old and new together.

cheers Repro steps:

1. 1. 1.

GraemeMalcolm commented 3 years ago

The .wav file is in the right folders in the repo. It was added 6 days ago - if you cloned it previously, you'd need to pull to get the latest files. I can't repro the issues you describe above with the current files in the repo.

GraemeMalcolm commented 3 years ago

Update - it seems this is an issue with the LODS hosted environment - the latest markdown files aren't being pulled into the instructions pane for some reason. Additionally, the LODS environment has stopped capturing the microphone (which is what necessitated the update in the first place) I've reported both issues to LODS, and they're working on it. Unfortunately there's nothing we can do from a content perspective - the files in this repo are up to date (and tested), but they're not currently getting pulled into the hosted lab environment. I'll leave this issue active until there's a fix for the LODS environment. In the meantime, your best bet is probably to have students follow the instructions directly from the site pages for this repo at https://microsoftlearning.github.io/AI-102-AIEngineer/ rather than the instructions pane in the hosted environment. Cc @lodsgithub

ZekeStarr commented 3 years ago

We (LODS) have updated the labs so that they fall in line exactly with what Graeme was expecting students to be seeing. In order to see this update, if your lab is currently running, you will need to refresh your lab window. Otherwise, when you resume any saved labs for AI-102, or launch a new lab, you will see the updated instructions.

We were not able to replicate the .wav files not downloading, but if you continue to see this, please don't hesitate to submit a ticket with our support so that we can evaluate the issue as it is ongoing.

On the microphone issue, we have engaged the 3rd party developers responsible for the gateway that is causing the loss of microphone utility. They are working on the issue, but we do not yet have an estimated time of resolution. Thank you Graeme, for accounting for this in your latest updates.

GraemeMalcolm commented 3 years ago

I think the point is that the WAV files ARE downloading, but the instructions pane isn't pulling in the latest instructions markdown file, so students have no idea what to do with them. As I understand it, the instructions pane should automatically pull in the latest markdown from GitHub; so any changes we make are automatically reflected there when a new instance of the lab starts (or the user refreshes the browser). That doesn't seem to be happening.

Cheers, G


Graeme Malcolm @graeme_malcolm


From: Zeke Starr @.> Sent: 13 April 2021 18:19 To: MicrosoftLearning/AI-102-AIEngineer @.> Cc: Graeme Malcolm @.>; State change @.> Subject: Re: [MicrosoftLearning/AI-102-AIEngineer] Lab 11 Using Wav file not in LODS (#59)

We (LODS) have updated the labs so that they fall in line exactly with what Graeme was expecting students to be seeing. In order to see this update, if your lab is currently running, you will need to refresh your lab window. Otherwise, when you resume any saved labs for AI-102, or launch a new lab, you will see the updated instructions.

We were not able to replicate the .wav files not downloading, but if you continue to see this, please don't hesitate to submit a ticket with our support so that we can evaluate the issue as it is ongoing.

On the microphone issue, we have engaged the 3rd party developers responsible for the gateway that is causing the loss of microphone utility. They are working on the issue, but we do not yet have an estimated time of resolution. Thank you Graeme, for accounting for this in your latest updates.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftLearning%2FAI-102-AIEngineer%2Fissues%2F59%23issuecomment-819153243&data=04%7C01%7CGraeme.Malcolm%40microsoft.com%7Ca30ebda3cc3f4d35a4cc08d8fee36494%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637539599892117162%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=SdnYl%2FDceY3sTDvf1GYknioN2eD0xvbs8pmL0TZVHjw%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACZZZ7WA25AXTZE4UXEHIKDTITUTFANCNFSM42337PIA&data=04%7C01%7CGraeme.Malcolm%40microsoft.com%7Ca30ebda3cc3f4d35a4cc08d8fee36494%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637539599892127157%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=zLANbDLlH8Kj9kdQe0WsASngmqj8mPnRJn96aldRrGo%3D&reserved=0.

ZekeStarr commented 3 years ago

Yep! That's what we were seeing too. We've re-worked the instructions so that that match your development setup, and sync with GitHub immediately (there can be up to a few minute delay, as we're relaying on the raw GitHub links). Outside of that, the only lingering issue should be the microphone passthrough, which we're getting traction on and hope to have resolved soon.

GraemeMalcolm commented 3 years ago

Closing because the instructions issue is resolved - awaiting fix for mic redirection issue