centreborelli / s2p

Satellite Stereo Pipeline
GNU Affero General Public License v3.0
207 stars 67 forks source link

reconcile readme install instructions #104

Closed shortwavedave closed 3 years ago

shortwavedave commented 3 years ago

While running through the readme instructions, this command: pip install -r ".[test]" produces this error: ERROR: Could not open requirements file: [Errno 2] No such file or directory: '.[test]'

This pull request updates the readme to match what the Dockerfile prescribes: pip install -e .

carlodef commented 3 years ago

Hi @shortwavedave, there is a typo in your command. The readme says pip install -e ".[test]", with -e, not -r.

The command is tested in github actions (see https://github.com/centreborelli/s2p/blob/master/.github/workflows/tests.yml#L18) and works fine.