Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add setup.py script #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add setup.py script #21

wants to merge 1 commit into from

Conversation

tim-fan
Copy link

@tim-fan tim-fan commented Aug 3, 2019

Hi there, thanks for the great package!

I forked to add a setup.py script, so I can now install the package via:
pip install git+https://github.com/tim-fan/GazeTracking.git

I'm making this pull request in case you want this change in your repo.

It was the first time I've written a setup.py file, so let me know if you want anything changed, for instance if you don't want your email in the package info, or if you'd like to start the version number at something other than 0.1.

I suppose a next step could be to upload to PyPI, although this is something I've never tried.

@hp77-creator
Copy link

Is it working on your system? Just asking. I haven't tried it on my system. Are there any system requirements to install the software?

@tim-fan
Copy link
Author

tim-fan commented Jun 10, 2020

Yep it works for me. I just tested it on bionic - I was able to run the example as follows:

python3 -m venv venv
source venv/bin/activate
pip install git+https://github.com/tim-fan/GazeTracking.git
wget https://raw.githubusercontent.com/tim-fan/GazeTracking/master/example.py
python example.py

(note it would be nice to include the example script in the package, so it is available at the command line after pip installing)

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.

2 participants