Skip to content

Feedback#1

Open
github-classroom[bot] wants to merge 30 commits into
feedbackfrom
main
Open

Feedback#1
github-classroom[bot] wants to merge 30 commits into
feedbackfrom
main

Conversation

@github-classroom
Copy link
Copy Markdown

@github-classroom github-classroom Bot commented Oct 9, 2023

👋! GitHub Classroom created this pull request as a place for your teacher to leave feedback on your work. It will update automatically. Don’t close or merge this pull request, unless you’re instructed to do so by your teacher.
In this pull request, your teacher can leave comments and feedback on your code. Click the Subscribe button to be notified if that happens.
Click the Files changed or Commits tab to see all of the changes pushed to main since the assignment started. Your teacher can see this too.

Notes for teachers

Use this PR to leave feedback. Here are some tips:

  • Click the Files changed tab to see all of the changes pushed to main since the assignment started. To leave comments on specific lines of code, put your cursor over a line of code and click the blue + (plus sign). To learn more about comments, read “Commenting on a pull request”.
  • Click the Commits tab to see the commits pushed to main. Click a commit to see specific changes.
  • If you turned on autograding, then click the Checks tab to see the results.
  • This page is an overview. It shows commits, line comments, and general comments. You can leave a general comment below.
    For more information about this pull request, read “Leaving assignment feedback in GitHub”.

Subscribed: @jkaszowski @ryanWh00 @suheylakeskin @Zip-Lip @Pastucha

github-classroom Bot and others added 21 commits October 9, 2023 21:12
* add draft of project charter

* update project charter

* latest updates

* add driving and idle time to requirements

---------

Co-authored-by: Suheyla Keskin <ksknshyl@gmail.com>
* cleanup READMEs

* add project logo
* Update README.md

* Update README.md

* Update README.md

* video stream

video stream on computer side with python code and opencv

* script on rpi

script on rpi that will run libcamera-vid ( which is the command to stream) all the time, and will not close connection after user close app on client side

* server

basic server that recive data sent by client and showing what the client send ( later to improve that it will have command handler )

* basic client

basic client side of the communication.
App can run stream and sending commands :)))))
POZDRO KUBA PATRZ JAK TATA JEST WIELKI

* Update client.py

I added the info about how to steer and right now commands can be send all the time to the server

* Update server.py

Handling reciving commands all the time from the client side, also sending commands forward to the uart port.

* Update client.py

* Update client.py

* Update client.py

---------

Co-authored-by: Jakub Kaszowski <jakub.kaszowski@outlook.com>
* add stm32 project

* Remove Debug/

* fixes
added exceptions handling and local variables
Co-authored-by: Paweł Rak <264355@student.pwr.edu.pl>
* Update client.py

* Update client.py

* Update client.py

* Update README.md

* Add files via upload

* Update client.py

* Update client.py
* Motors first try

* fixes

* Remove drivers and update .ioc
* Fix handling keys

In some cases (when camera is not connected), the code fails on self.cap.read() and does not send commands on keypress

* Fix commands format

Force all numbers sent to be integers
* Motors first try

* fixes

* Remove drivers and update .ioc

* Add template for servos

* Add servos to .ioc

* Update .gitignore

* Add unit tests and parsing for servos

* Update README
ghost pushed a commit that referenced this pull request Jan 16, 2024
jkaszowski and others added 7 commits January 17, 2024 13:02
* new YOLO detection

* Names of objects (#1)

* fixing opencv in client.py

add code from the branch Ryan-Fixed

* some fixes

* Configuration Files

---------

Co-authored-by: ryanWh00 <269580@student.pwr.edu.pl>
Co-authored-by: Mikołaj Pastucha <67973699+Pastucha@users.noreply.github.com>
* Add low level software section draft

* Add Mikolaj's part
* Add final version of the report

* Remove old report template

* remove unused pdf

* Cleanup
* try to fix opencv

* inversion of text in bounding box

* Update client.py

* Update stream.sh

---------

Co-authored-by: ryanWh00 <99709108+ryanWh00@users.noreply.github.com>
@JanuszJakubiak
Copy link
Copy Markdown

Proposed grade: 5.0

Issues:

  • The robot is a (fairly) complex system, consisting of several components. The structure of the report misses a presentation of the general idea of the system as the whole, before separate components are described. A reader not familiar with the project will have a hard time at the beginning of the report not knowing how the components fit to the system. It could be solved by an additional subsection in sec.1 presenting the idea (with components and their connections/dependence) and more specifically the functionalities.

Other (remarks for future):
With the perspective of future Engineering Thesis the drawbacks of the report
are (in most sections):

  • missing proper literature review with analysis of the other works and references to the original sources of solutions in the main text
  • the conclusions are based on observations that were not documented in the text/experimental section

@JanuszJakubiak
Copy link
Copy Markdown

Corrections accepted.
Final grade: 5.5

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants