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

incorrect README.md > upload_with_preprocessing.py > env. vars set #226

Closed
ProBackup-nl opened this issue May 7, 2018 · 3 comments
Closed

Comments

@ProBackup-nl
Copy link

ProBackup-nl commented May 7, 2018

Basic information

  • Version: ?
  • System: Mac OS X 10.9.5 + zsh shell
  • Capture Device: n/a

Steps to reproduce behavior

  1. read README.md for upload_with_preprocessing.py
  2. it states:

Requires environment variables set: MAPILLARY_USERNAME, MAPILLARY_PERMISSION_HASH, MAPILLARY_SIGNATURE_HASH from

  1. define these 3
  2. check with echo $VAR that each one is set correct
  3. run ./upload_with_preprocessing.py

Expected behavior

No error

Actual behavior

You are missing one of the environment variables MAPILLARY_USERNAME, MAPILLARY_EMAIL, MAPILLARY_PASSWORD, MAPILLARY_PERMISSION_HASH or MAPILLARY_SIGNATURE_HASH. These are required.

Even with all 5 set (not exported), this error message still occurs.

@jernejaMislej
Copy link
Contributor

hi @ProBackup-nl
are you using the latest version of master?
although email and password are not required for image preprocessing, they are required for authentication when uploading. have you tried with all 5 exported?
i apologize for the inconvenience, we are working on improving the authentication process to avoid exporting all these variables and will update the readme when the changes are released

@ProBackup-nl
Copy link
Author

Have tried with all 5 exported. Still an error.

@jernejaMislej
Copy link
Contributor

hi, just letting you know we have released a new version of the tools, where authentication is made simpler and easier, so only the user name has to be specified as an argument and nothing needs to be exported in the os environment. Please try to use the new tools, instructions and details in the README and let me know if you have any more issues, otherwise i will close this

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

No branches or pull requests

2 participants