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

Add Camera Capture Basics Tutorial #890

Merged
merged 4 commits into from Jan 6, 2022

Conversation

PopThosePringles
Copy link
Collaborator

Part 1 of the Camera Capture series. Refactored the whole tutorial into parts. Changed some wording to make it a bit more beginner friendly in the last 3 sections, moved things around so the stuff you need to know to get into camera capturing is above the 3 optional sections that explain and show of how to use certain functions.

So we decided to split the tutorial into parts. Icon Generator will be part 2, and Photo Booth will be part 3. Part 2 needs to be refactored to be a standalone tutorial.

Part 1 of the Camera Capture series. Refactored the whole tutorial into parts. Changed some wording to make it a bit more beginner friendly in the last 3 sections.
@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2022

Images automagically compressed by Calibre's image-actions

Compression reduced images by 23.1%, saving 55.47 KB.

Filename Before After Improvement Visual comparison
src/img/CameraCapture/Basics/basics_camera_properties.png 85.30 KB 65.07 KB -23.7% View diff
src/img/CameraCapture/Basics/basics_create_camera.png 31.32 KB 24.61 KB -21.4% View diff
src/img/CameraCapture/Basics/basics_script_properties.png 88.92 KB 68.42 KB -23.1% View diff
src/img/CameraCapture/Basics/basics_script.png 34.28 KB 26.26 KB -23.4% View diff

1310 images did not require optimisation.

@PopThosePringles PopThosePringles added the needs review Ready for review label Jan 5, 2022
@netlify
Copy link

netlify bot commented Jan 5, 2022

✔️ Deploy Preview for manticore-docs ready!

🔨 Explore the source changes: 9f4d3ff

🔍 Inspect the deploy log: https://app.netlify.com/sites/manticore-docs/deploys/61d5ef9f0d28a00007773073

😎 Browse the preview: https://deploy-preview-890--manticore-docs.netlify.app

@netlify
Copy link

netlify bot commented Jan 5, 2022

✔️ Deploy Preview for manticore-docs ready!

🔨 Explore the source changes: dffc7c1

🔍 Inspect the deploy log: https://app.netlify.com/sites/manticore-docs/deploys/61d6c2625ba4a500075bcf53

😎 Browse the preview: https://deploy-preview-890--manticore-docs.netlify.app

Copy link
Contributor

@DoubleA-MG DoubleA-MG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The first half seems beginner friendly, but the second half does seem a bit out of place knowledge-level wise. It might be better to move the memory management and capture release sections to the future lessons.

src/tutorials/camera_capture_basics.md Show resolved Hide resolved
@PopThosePringles
Copy link
Collaborator Author

PopThosePringles commented Jan 6, 2022

The first half seems beginner friendly, but the second half does seem a bit out of place knowledge-level wise. It might be better to move the memory management and capture release sections to the future lessons.

Yeah. The problem is that it's handy to know about releasing and refreshing. The refreshing is more important as it opens up more possibilities. We could just take these and put them into the UI ref and link them. It depends, hmm.

Will talk more about this in standup and see what would be best.

I felt that before it was like the owl drawing. Draw a circle,  now you have an owl. So I broke it up a little, just in case the user wants to follow along and create the same scene.
@StanzillaManticore StanzillaManticore merged commit ad61d61 into development Jan 6, 2022
@StanzillaManticore StanzillaManticore deleted the foo-camera-capture-basics-tutorial branch January 6, 2022 23:01
StanzillaManticore pushed a commit that referenced this pull request Mar 28, 2022
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review Ready for review
Development

Successfully merging this pull request may close these issues.

None yet

3 participants