lbs-iitkgp / opensoft18

Repository for the Gold-winning OpenSoft 2018 submission 🎉
GNU General Public License v3.0
12 stars 7 forks source link

Add installation script #36

Closed athityakumar closed 6 years ago

athityakumar commented 6 years ago

Writing a shell script to install all the dependencies, would make deployment easier.

cd frontend && yarn
cd ../backend
pip3 install -r requirements.txt
python3 -m nltk.downloader all
rudrasohan commented 6 years ago

@athityakumar are we using both python2 & python3 ?

athityakumar commented 6 years ago

I'm using python3 (3.6.1).

@ghostwriternr - Please don't tell me that the codebase is partly python2 and partly python3? 😢

rudrasohan commented 6 years ago

I'm just worried about the OpenCV part. People generally use OpenCV packaged with ROS which is in python2.7

ghostwriternr commented 6 years ago

I don't what we've been using so far, but we should stick to Python3.

athityakumar commented 6 years ago

Well, there is only so much that __future__ can quick-fix between python2 & python3. Quick-fixing dependencies between pip & pip3 is sadly beyond its scope. :trollface:

@rudrasohan - We definitely don't need ROS for our task, and I think OpenCV with python3 should work just fine. Though, can you confirm that?

ghostwriternr commented 6 years ago

Ohh yep. ROS isn't needed. (Source: I don't have ROS)

On Sat, 24 Mar 2018 at 8:31 AM, Athitya Kumar notifications@github.com wrote:

Well, there is only so much that future can quick-fix between python2 & python3. Quick-fixing dependencies between pip & pip3 is sadly beyond its scope. [image: :trollface:]

@rudrasohan https://github.com/rudrasohan - We definitely don't need ROS for our task, and I think OpenCV with python3 should work just fine. Though, can you confirm that?

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/lbs-iitkgp/opensoft18/issues/36#issuecomment-375841944, or mute the thread https://github.com/notifications/unsubscribe-auth/AJjyv4e8DSE0JmjLf965-Z0LkAxJeTxsks5thbcZgaJpZM4S3s_Y .

thealphadollar commented 6 years ago

Done.