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

Provides the ability to run in environment execution where multiproce… #111

Merged
merged 1 commit into from
Jan 30, 2018

Conversation

michelorengo
Copy link

Importing the tdqm module (_tqdm.py creates 2 global locks) in environments where multiprocessing locks are not implemented raises an OSError. For instance, running papermill in an AWS Lambda will fail because the AWS Lamdba execution environment does not have /dev/shm (shared memory for processes) support.
This suggested PR checks whether tdqm can be imported or not and prevent the use of the progress bar if the module can't be imported.

…ssing locks are not implemented, e.g. AWS Lambda
@codecov
Copy link

codecov bot commented Jan 29, 2018

Codecov Report

Merging #111 into master will decrease coverage by 0.05%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
- Coverage   63.24%   63.19%   -0.06%     
==========================================
  Files           8        8              
  Lines         917      921       +4     
==========================================
+ Hits          580      582       +2     
- Misses        337      339       +2

@rgbkrk rgbkrk requested a review from MSeal January 30, 2018 16:54
@MSeal MSeal merged commit 748020d into nteract:master Jan 30, 2018
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.

3 participants