search

airavata-courses / alpha

Team Alpha Project Repo
0 stars 2 forks source link

Peer Review - Flask version conflict with Python #15

Closed karanrk closed 6 years ago

karanrk commented 6 years ago

Hello Team,

1.Nice to see ansible set up for MAC users (Looks like it is high time to buy MAC soon) :)

  1. Since i don't own a mac, I tried to start the News microservice implemented in python with Flask. To give a background of the issue I am running into, I have latest Flask already installed on my machine, but looks like my system is not allowing me to install the flask version that you specified in the requirements.txt file. ( pip2 works just fine, but not pip3) also when i start the flask server, i get no module named flask error.

Best Regards, Karan

siddharthpathak commented 6 years ago

There might be version conflict with flask. Generally, python modules have clashes with different versions, mostly because of people using both Python 2.x and 3.x. I would suggest to make a virtual env using Python 3, and then install all the Python modules in that using requirements.txt. Once that is done you can run the flask server.

karanrk commented 6 years ago

Figured that out thanks, your virtual env activation step must be before pip install requirements.txt. Step 5 must be after step 2 on the News microservice Wiki page.

The mvn package clean step fails in test1GetDataFromThirdParty, i am guessing it is because we have not provided valid API_KEY.

siddharthpathak commented 6 years ago

Fixed the step order in wiki. Thanks for finding it out.

You can get the keys from news and weather

hardikrakholiya commented 6 years ago

@karanrk let me know if you need any help. Else if this is resolved please close the issue.

karanrk commented 6 years ago

Tried with valid API_keys and works fine.Good job guys.