search

AryaChakraborty / articulus_rag

25 stars 33 forks source link

Dockerized & requirements.txt fixed #58 #15 #62

Closed DebasisX closed 4 months ago

DebasisX commented 4 months ago

This is the final commit after resolving the conflicts in requirements.txt and have also added the Dockerfile with environment variables in the .env file which is loaded in the app.py.

AryaChakraborty commented 4 months ago

@DebasisX I think we already have .env.example to mock the environment variables

additionally, I don't see any command that would export the variables in the environment. Let me know if i missed something.

DebasisX commented 4 months ago

I just added a better comment to the mock .env.example file.

As for the Dockerfile, The .env is copied into the container and when the app.py is launched the python code can access the .env variables so there's no need to export. I also checked out the app.py it loads .env by default so I didn't had to add that.

AryaChakraborty commented 4 months ago

@sbk2k1 can you please review the commits?

DebasisX commented 4 months ago

@AryaChakraborty So there's an issue, regarding the label as my GSSoC points aren't updated.

The label should be marked as level3 i.e. there shouldn't be any space between label and the corresponding numbers. Kindly help me by reassigning the labels. Thank you.