search

Fireboltz / Psychic-CCTV

A video analysis tool built completely in python.
43 stars 12 forks source link

Fix Requirement.txt #8

Closed Vishwaak closed 3 years ago

Vishwaak commented 4 years ago

Summary

There are a lot of redundant modules in the requirement.txt. The one not necessary for the projects can be removed.

saisiddhant12 commented 4 years ago

can I take this up ?

Vishwaak commented 4 years ago

Yeah feel free, to work on this issue :smile:

ShehzadAhm commented 4 years ago

Hi @Vishwaak, can I work on this issue?

saisiddhant12 commented 4 years ago

@ShehzadAhm am working on it, feel free to look at other issue on pyqt

ShehzadAhm commented 4 years ago

@saisiddhant12 Ok no problem.

saisiddhant12 commented 4 years ago

@Vishwaak I see there is a problem in usual installation of pyaudio for python 3.8, do you want to create a separate issue for that ?

Vishwaak commented 4 years ago

It would be better if we can stick around with Python 3.7 for the project as there are few other modules that currently don't have support for the Python 3.8.

saisiddhant12 commented 4 years ago

Got it, I will validate the environment using python 3.7

saisiddhant12 commented 4 years ago

@Vishwaak I believe packages like tkinter cant be installed using pypy packages rather we need to install on the platform's package manager,

saisiddhant12 commented 4 years ago

@Vishwaak I was able to create a PR for the requirements and the issue is mentioned over here https://github.com/Fireboltz/Psychic-CCTV/pull/9

The part where I had to use package mnager instead of pip are as follows tkinter: python-dev portaudio19-dev