wvsu-cict-code / thesis-package

College's guidelines in packaging your thesis application.
2 stars 4 forks source link

Sarabia et. al. Package Evaluation #11

Closed cictapps closed 2 years ago

cictapps commented 2 years ago

Title:

Face Mask and Physical Distancing Detection System Using Computer Vision

Members:

Hugo Leroy D. Chavez, Flora May D. Gicanal, Christian T. Sarabia, Marianne Therese E. Tunggak

geniusqwerty12 commented 2 years ago

Started checking the user manual and tried to follow the instructions on how to run the program, so far these are the things I encountered:

User Manual:

  1. Page 4, the command pip install -r requirement.txt, the file name should be requirements.txt.

Errors:

  1. ERROR: Could not find a version that satisfies the requirement wintoast==0.9 (from versions: 0.1, 0.2) ERROR: No matching distribution found for wintoast==0.9 this is an error I encountered when trying to run the install modules/packages command. Upon checking the codes, I think you used win10toast instead of wintoast (which is currently on the requirements.txt), changing it to win10toast==0.9 solves the issue.

  2. Also, the module plyer was not found on the requirements.txt, please add as main.py uses it.

  3. Following what the user manual, I've created the yolo-coco folder and pasted the weights, however when I tried to execute the program, an error or two was encountered [Errno 2] No such file or directory: 'yolo-coco/coco.names' net = cv2.dnn.readNetFromDarknet(configPath, weightsPath) After getting it from on of the members, I was able to run the program. Please put the missing files coco.names and yolocv3.cfg in the repo, preferably inside the yolo-coco folder, so that the user will just download the weights

As for the manuscript, I found no problems.

Sturg25 commented 2 years ago

The mentioned concerns has been addressed:

The Manual and the Readme.md has also been edited to apply changes to the installation instructions of the project.

geniusqwerty12 commented 2 years ago

Just confirmed that all of the concerns have been addressed on the repository. @Sturg25 let me know if you can or can't create the exe file.

geniusqwerty12 commented 2 years ago

Passing this to @cictapps @mjsolidarios @regincabacas for approval