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

Luigi Contrib Modules where moved #41

Closed
xaedes opened this issue Sep 19, 2017 · 2 comments
Closed

Luigi Contrib Modules where moved #41

xaedes opened this issue Sep 19, 2017 · 2 comments

Comments

@xaedes
Copy link

xaedes commented Sep 19, 2017

See this luigi commit:
spotify/luigi@781613e

breaking: Remove deprecated contrib redirections (#2181)

After this commit the following imports will no longer work

 import luigi.s3
 import luigi.postgres

Instead use

 import luigi.contrib.{s3,postgres}>  

Results in following error:

Traceback (most recent call last):
File "workflow.py", line 2, in
import sciluigi as sl
File "/usr/local/lib/python3.5/dist-packages/sciluigi/init.py", line 11, in
from sciluigi import dependencies
File "/usr/local/lib/python3.5/dist-packages/sciluigi/dependencies.py", line 7, in
from luigi.postgres import PostgresTarget
ImportError: No module named 'luigi.postgres'

xaedes pushed a commit to xaedes/sciluigi that referenced this issue Sep 19, 2017
Updated import path of contrib modules
@xaedes xaedes mentioned this issue Sep 19, 2017
@samuell
Copy link
Member

samuell commented Sep 19, 2017

Should be fixed with #40 that was now merged.

@samuell
Copy link
Member

samuell commented Sep 19, 2017

Closing now, please report if any remaining errors!

@samuell samuell closed this as completed Sep 19, 2017
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