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 'six' to requirements. #21

Merged
merged 1 commit into from
Dec 28, 2015

Conversation

solongordon
Copy link
Contributor

A user reported issues importing the client because he didn't have the
'six' module installed. I added this to the install requirements in
setup.py.

A user reported issues importing the client because he didn't have the
'six' module installed. I added this to the install requirements in
setup.py.
@solongordon
Copy link
Contributor Author

@jimreesman for review

@jimreesman
Copy link
Contributor

i didn't check if other requirements have crept in - would you pls double-check? tx. then it's gtg.

@solongordon
Copy link
Contributor Author

Good question, I'll check.

@solongordon
Copy link
Contributor Author

Nope, looks like it's just six. I don't have write access anymore so I'll leave the merge to you.

jimreesman added a commit that referenced this pull request Dec 28, 2015
@jimreesman jimreesman merged commit c5fba62 into Clarifai:master Dec 28, 2015
josh-tepper pushed a commit that referenced this pull request Aug 11, 2023
…etries (#21)

* clear_cache function

* authorized stub

* formatting

* edit tests

* update getattr

* retry stub

* batched sequences

* move files

* parallel list inputs

* init file

* use abc for type checks

* imports in init

* use authorized client and remove seprate metadata var

* edit readme

* revert parallel sequence call edits

* change mock order

* remove mock from error class

* doc strings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants