Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

Yue playground #43

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Yue playground #43

wants to merge 29 commits into from

Conversation

assassins567
Copy link
Collaborator

needs demands from back-end at forks and crosses

@Luyanda-Mlangeni
Copy link
Collaborator

Could you remove all your local files from obj?

@assassins567
Copy link
Collaborator Author

Could you remove all your local files from obj?

ok, I do it now

@HClassen
Copy link
Collaborator

HClassen commented Jul 7, 2020

It would also be good if you split up the one big function into three smaller functions:

  • image preprocessing -> should return the black/white image
  • finding the white line
  • calculating the wheel velocities

this will make testing and integration easier

@HClassen
Copy link
Collaborator

HClassen commented Jul 8, 2020

Please again remove all files in test/ada/obj as well as in external/controller/obj

Copy link
Collaborator

@HClassen HClassen left a comment

Choose a reason for hiding this comment

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

There are still some binary files in you branch which should be removed. Also there a some merch conflicts to be resolved. After that is done the branch can be merged.

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

Successfully merging this pull request may close these issues.

3 participants