-
Notifications
You must be signed in to change notification settings - Fork 45
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
Allow to disable importing #492
base: master
Are you sure you want to change the base?
Conversation
This will not work this way as most modules initialize data in the import method. At least this needs to run for the motis/modules/nigiri/src/nigiri.cc Lines 473 to 481 in 092abec
motis/modules/nigiri/src/nigiri.cc Lines 500 to 522 in 092abec
Same for the Lines 347 to 352 in 092abec
I guess the only option is to have a additional parameter to the |
I guess another option would be to split the import functionality into preprocessing the data and actually loading the data. My plan is to address this and many other issues mentioned in #452 (such as a stable API + versioning, running MOTIS without the |
I haven't thoroughly tested this yet