Skip to content

Update README.md - #79

Open
ofcoursenp wants to merge 2 commits into
moest-np:mainfrom
ofcoursenp:main
Open

ofcoursenp wants to merge 2 commits into
moest-np:mainfrom
ofcoursenp:main

Conversation

@ofcoursenp

Copy link
Copy Markdown

Added library installations

Added library installations
@thesushilsharma

Copy link
Copy Markdown

At the moment

The required libraries are assumed to be installed directly using python3, so there's no need to install them from the requirements.txt file. You can run the python script immediately without any additional library installation for basic functionality.

@ofcoursenp

ofcoursenp commented Apr 26, 2024 via email

Copy link
Copy Markdown
Author

@ofcoursenp ofcoursenp left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed Installation of pip -r requirements.txt which was left unnoticed

@LuluW8071

Copy link
Copy Markdown
Contributor

Don't think this is necessary since it is basic command to install pip packages.

@ofcoursenp

Copy link
Copy Markdown
Author

Don't think this is necessary since it is basic command to install pip packages.

not everyone knows but still its just a contribution they want to add it or they can close the pull request

@sapradhan

Copy link
Copy Markdown
Collaborator

i think purpose of this section is to provide high level steps involved. exact commands to run including the one proposed can be found in CONTRIBUTION.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants