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

Other notes on labs #456

Closed
BravoBaldo opened this issue Sep 17, 2024 · 0 comments · Fixed by #460
Closed

Other notes on labs #456

BravoBaldo opened this issue Sep 17, 2024 · 0 comments · Fixed by #460
Labels
improvement Improve existing content

Comments

@BravoBaldo
Copy link
Contributor

(Please Note: as always, some parts are personal opinions!)

In "seeed/xiao_esp32s3/kws/kws.qmd":
Search "which you can find on the" --> insert a spacebefore.
Search "YES. We got" --> "YES, we got"

In "seeed/motion_classification/motion_classification.qmd":
Search "m/s2" --> 2 must be squared (at least in text part)
"Note that A message " ---> A in lowercase
"CLI Data Forwarder t" --> remove " t" and append it to next word
"Palettes being handled by" --> in the second occurrence is missed "Fork-Lift)"
"Now it is time for magic˜!" --> it appears a strange character before '˜ ' because it is not "!" but the character 0xCB9C21

In "labs\raspi\raspi.qmd"
"**Raspberry Pi :" --> remove space before ""

In "labs\raspi\setup\setup.qmd"
"Key features:" --include the s in the title
"Raspberry Pis checks" remove one of the two s.
Search "rasivid" --> raspivid" (P)
In the .qmd, search "The original 5-megapixel model was": in this sentence there are 3 links... and 3 problems:
1) the first link is not well formatted: insert a space before "in 2013"
2) The First link is "released" but not the name, the second one is the name of the module. Please review
3) The third one is partially broken (contain a space!)...please update
Search "Any camera module" ---> in the sentence: "onfiguration.txt" --> "Configuration.txt" (prepend a C)
Search "libcamera", after the first occurrence append a space

In "labs\raspi\image_classification\image_classification.qmd"
Search "numpy.int8" maybe must be included in previous code ?
Search "create a new Python script combining" this is the point 2 in a list.... missing point 3...or wrong sequence!!
Search "goes from 0 to 256." --> "goes from 0 to 255."
Search "(int8 quantized model" --> missing closing parenthesis
Search "Rasp-" ---> 4 occurrences, replace with "Raspi-"

@profvjreddi profvjreddi added maintainence quarto/git bookkeeping improvement Improve existing content and removed maintainence quarto/git bookkeeping labels Sep 18, 2024
@profvjreddi profvjreddi linked a pull request Sep 19, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improve existing content
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants