w3c / wot-cg

Web of Things Community Group
https://www.w3.org/community/wot/
Other
16 stars 10 forks source link

10th tutorial video review #120

Open handealptekin opened 8 months ago

handealptekin commented 8 months ago

Please use the following for reviews: https://murf.ai/share/loroxo5j

netlify[bot] commented 8 months ago

Deploy Preview for whatiswot failed.

Name Link
Latest commit fc9a0e0027c418eaa4d188267f9e5ae6dda94250
Latest deploy log https://app.netlify.com/sites/whatiswot/deploys/655d14cf0eafcb0008e01188
FadySalama commented 8 months ago

I am not sure about the pronunciation of DECT ULE and Wi-SUN (See 2:00). I think ULE should at least be spelled. @egekorkan Do you know how they are usually pronounced?

egekorkan commented 7 months ago

I would say we simply remove those two examples. They are a bit corner case and it would be difficult to back them up with TDs later on.

handealptekin commented 7 months ago

All of the requested changes have been made except the "wiggling" issue. I will proceed with the further feedback on that. Video can be reviewed from the same link. I also added the new outro to the end.

egekorkan commented 7 months ago

The important feedback is addressed. One pronunciation and animation fix is still needed. Afterwards, I will ask feedback from @relu91 .

handealptekin commented 7 months ago

I delayed the wiggle until 00:54 and revised the pronunciation of PCs (however I am not sure of if it is okay). Please review the updated link.

egekorkan commented 7 months ago

The wiggle should be a bit later but the animation is too fast (or talking is too slow). The sentence should finish before wiggling and then the transition to the inside of the home starts.

handealptekin commented 7 months ago

I added more delay to the wiggle and updated the video.

egekorkan commented 7 months ago

All my concerns are addressed, we will wait for @relu91 review before merging.

egekorkan commented 6 months ago

A generic comment about use cases. I video explains correctly the different use cases but it fails to convey what WoT can do for the different domains. It is like we are explaining well- known IoT use cases. We are listing different protocols in the description why don't explicitly say that this diversity make difficult to create one-fit-all solutions?

So this was inspired from the architecture document since it also starts in this generic fashion. So I propose to do the following changes: