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

Added pwd to python's path when running on cluster. #1797

Merged
merged 1 commit into from
Aug 2, 2016

Conversation

brcopeland
Copy link
Contributor

Description

When submitting to the cluster, sge.py now passes the present working directory to sge_runner.py. sge_runner.py uses this new parameter to append to the python path.

Motivation and Context

These changes allow the user to properly import scripts in the same directory as the luigi script, even if they're not in the user's normal $PYTHONPATH. Without these changes, attempting to do this results in an ImportError.

Have you tested this? If so, how?

I've been using these changes for several months without issue.

@mention-bot
Copy link

@brcopeland, thanks for your PR! By analyzing the annotation information on this pull request, we identified @jfeala to be a potential reviewer

@Tarrasch Tarrasch merged commit b860fbd into spotify:master Aug 2, 2016
@brcopeland brcopeland deleted the pwd_fix branch August 3, 2016 22:04
This was referenced Jun 29, 2022
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