Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intro to Operators - Lesson 3 #75

Open
jonasmit opened this issue Nov 16, 2021 · 9 comments
Open

Intro to Operators - Lesson 3 #75

jonasmit opened this issue Nov 16, 2021 · 9 comments

Comments

@jonasmit
Copy link
Contributor

No description provided.

@djschny
Copy link

djschny commented Nov 18, 2021

After reviewing lesson 3 recording, see the following for items:

  1. The camera video box has the same problem we saw in the first version of lesson1 where it extends past the top of the slides and in from the right edge.
    Screen Shot 2021-11-18 at 12 20 39 PM
  2. The diagram in slide 2 should be reworked to look more professional.
  3. The video appears unnatural as it looks like in between each sentence there is a hard cut to a different video clip.

I think it's OK for launch, but we should add an item to the backlog to redo the video when time permits.

@djschny
Copy link

djschny commented Nov 22, 2021

I've updated the diagram in the lesson 1 deck which can also be used here:

Screen Shot 2021-11-22 at 10 35 14 AM

@jonasmit
Copy link
Contributor Author

@lxie123 - recording this week by Dec 1

@lxie123
Copy link
Contributor

lxie123 commented Nov 30, 2021

I've updated the diagram in the lesson 1 deck which can also be used here:

Screen Shot 2021-11-22 at 10 35 14 AM

Hmm not much difference here vs original, just some highlight of the bottom right box. Also, the original matches all her other slides more.. I prefer the original for this chapter.

@djschny
Copy link

djschny commented Nov 30, 2021

The lines are all organized and match up.

@djschny djschny added the Lesson label Nov 30, 2021
@lxie123
Copy link
Contributor

lxie123 commented Dec 2, 2021

@djschny
Copy link

djschny commented Dec 3, 2021

Overall the content and presentation looks good. A few things that I made note of:

  • The video appears to be zoom'd out quite a bit so it's hard to see the speaker
  • The video quality appears to be low, for example it's hard to read the text in the code snippets on the slides due to pixelation
  • The diagrams are still the original ones that look unprofessional

@eitansuez
Copy link

i can confirm that the resolution of the slides is low. it's most noticeable on the "declarative configuration" slide (minute 4:00).

@lxie123
Copy link
Contributor

lxie123 commented Dec 8, 2021

New video uploaded with zoom fix and resolution fix.
https://drive.google.com/drive/folders/1yX3KhcinrtMGt2k4GsaNfv9PfWxUCxqp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants