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

/bin/bash: line 1: plastimatch: command not found #46

Open
TrinhNguyenP opened this issue Jan 12, 2024 · 5 comments
Open

/bin/bash: line 1: plastimatch: command not found #46

TrinhNguyenP opened this issue Jan 12, 2024 · 5 comments

Comments

@TrinhNguyenP
Copy link

Hi IDC,

I am following the tutorial named lung_nodules_demo.ipynb located in IDC-Tutorials/notebooks/deprecated.
I encounter an error as below:
Screenshot 2024-01-12 at 11 18 45 AM
Can you please guide me how to fix it?

Thanks,
Trinh

@fedorov
Copy link
Member

fedorov commented Jan 16, 2024

@TrinhNguyenP we fixed that issue for another notebook that you can see here: #30.

@denbonte is the developer of the notebook that doesn't work for you, and the one who identified the solution for plastimatch installation. Would you be willing to update this notebook as well Dennis?

@denbonte
Copy link
Contributor

denbonte commented Jan 16, 2024

Thanks for opening this issue @TrinhNguyenP.

@fedorov I will look into the notebook ASAP!

@denbonte
Copy link
Contributor

In the meantime, @TrinhNguyenP, since this will require a bit of work on our side (due to the deprecation, for the current Google Colab, of multiple components we used in that notebook), you might be interested in taking a look at this repository - which stores a series of notebooks we wrote, running the classification model from Hosny et al. on IDC data [1][2][3].

Let me know if I can help with anything else - I will try to fix the notebook you linked in the following days!

@TrinhNguyenP
Copy link
Author

In the meantime, @TrinhNguyenP, since this will require a bit of work on our side (due to the deprecation, for the current Google Colab, of multiple components we used in that notebook), you might be interested in taking a look at this repository - which stores a series of notebooks we wrote, running the classification model from Hosny et al. on IDC data [1][2][3].

Let me know if I can help with anything else - I will try to fix the notebook you linked in the following days!

Hi @denbonte,
I was able to install plastimatch and pyplastimatch. However, I am comimg up with another error with this line:
model = keras.models.model_from_config(model_json)
This is from the section Data processing, which loads the pre-trained weights for DeepPrognosis and run the last part of the pipeline.). I will shoot you a message if I can’t resolve the error following the suggestion from @deepakri201.
Thanks,
Trinh

@denbonte
Copy link
Contributor

Hey @TrinhNguyenP,

I was able to install plastimatch and pyplastimatch.

Awesome!

However, I am comimg up with another error with this line:
model = keras.models.model_from_config(model_json)
This is from the section Data processing, which loads the pre-trained weights for DeepPrognosis and run the last part of the pipeline.).

Yes. Unfortunately, Colab dropped the support for Tensorflow 1.x - which means that many models trained with TF 1.x (or Keras based on TF 1.x) were left in this state.

The way we solved this, as you can see in the repo I linked to you above, was by converting the Deep Prognosis model into the open ONNX format and using the ONNX engine to run the inference phase. We made the model available as part of the idc-radiomics-reproducibility repository, and have examples of how to run that in the notebooks.

since this will require a bit of work on our side (due to the deprecation, for the current Google Colab, of multiple components we used in that notebook),

This is exactly what I was planning to do, by the way. Clean up the notebook, pull the 22.04LTS binaries for Plastimatch, switch to the ONNX implementation of the model, and solve any other dependency issues. I'm still planning to tackle that whenever I have a moment, but if you want to give it a try following the other notebooks, do let me know how it goes!

Cheers,
Dennis.

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

3 participants