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

Reorganize the repo #13

Open
srikarg89 opened this issue May 26, 2020 · 0 comments
Open

Reorganize the repo #13

srikarg89 opened this issue May 26, 2020 · 0 comments
Assignees
Labels
Organization Something should be moved or deleted

Comments

@srikarg89
Copy link
Member

  • There's a bunch of branches and they're naming scheme is confusing. I'm thinking 3 branches: master, detection, and classification.
  • There's a bunch of folders in each branch. Ideally, there should be one folder for each method (HaarCascade/, ContourDetection/), and a main.py file that can import from any folder as needed. For example in the classification branch, each folder could have a file w/ a def classify(img): function, and the main.py file can simply import the classify function from the folder you want to test and call that. This way, there could also be a helpers/ folder which contains basic CV methods that are used by all of the other folders in order to reduce redundancy and make the code a lot cleaner.
  • We should use a requirements.txt file and a venv to make sure everyone's library versions are the same.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Organization Something should be moved or deleted
Projects
None yet
Development

No branches or pull requests

5 participants