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

Execution fails if the notebook imports a file #25

Open
mosoriob opened this issue May 7, 2021 · 3 comments
Open

Execution fails if the notebook imports a file #25

mosoriob opened this issue May 7, 2021 · 3 comments

Comments

@mosoriob
Copy link
Contributor

mosoriob commented May 7, 2021

I had a notebook main.ipynb that imports a function from the file inference.py

The script generated main is going to fail because the ipython2cwl doesn't copy the inference.py file

#!/usr/bin/env ipython
"""
def main(cameraType, siteID, modelPath):
    import numpy as np
    import inference, hpwren
$ ls
main
@mr-c
Copy link
Member

mr-c commented May 10, 2021

Dear @mosoriob , thanks for this report

Please try adding something like local_script: 'CWLFilePathInput' = 'inference.py' towards the beginning of each cell that needs inference.py?

By the way, if you can share your notebook then that will make it easier to help you.

@mosoriob
Copy link
Contributor Author

Hi @mr-c

The notebook is available at
https://github.com/iperezx/edge-plugin-smokedetect

A full description of the issue is here

mintproject/mic#344

@mosoriob
Copy link
Contributor Author

I don't like the idea of exposing the inference.py as input because the user who runs the model is not always the author.

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