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

null pointer exception in checkParameters() of SegmentationJob #9

Closed
ThorstenFalk opened this issue Jan 30, 2019 · 4 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@ThorstenFalk
Copy link
Collaborator

Issue detail: When runs segmentation job using BF example dataset and pretrained model provided by the paper, exception occurs:

Exception in thread "Thread-5" java.lang.NullPointerException
at de.unifreiburg.unet.SegmentationJob.checkParameters(SegmentationJob.java:390)
at de.unifreiburg.unet.SegmentationJob.getParameters(SegmentationJob.java:438)
at de.unifreiburg.unet.SegmentationJob.run(SegmentationJob.java:661)

log file indicates:

$ /home/daveb/unet-seg/bin/caffe_unet
$ /home/daveb/unet-seg/bin/caffe_unet
$ caffe_unet check_model_and_weights_h5 -model /home/daveb/Downloads/caffemodels/2d_cell_net_v0.modeldef.h5 -weights /home/daveb/Downloads/2d_cell_net_v0.caffemodel.h5 -n_channels 1 -gpu 0

Installation and OS details:

Build from source (using instructions from project website);

No backend used (running caffe_unet in terminal correctly shows usage information)

Ubuntu 18.04 LTS, CUDA 9.0 CuDNN 7

@ThorstenFalk ThorstenFalk added the bug Something isn't working label Jan 30, 2019
@ThorstenFalk ThorstenFalk self-assigned this Jan 30, 2019
@ThorstenFalk
Copy link
Collaborator Author

The checkParameters() function grew too big. Refacturing required to make it more maintainable. It looks like the ProcessResult, that is initialized with null is never set in the function in this case.

@ThorstenFalk
Copy link
Collaborator Author

checkParameters() function simplified. Exceptions and failing processes now immediately make the check fail which should as a side-effect fix this bug.

@yinawang28
Copy link

Hi Thorsten, do you mind share a description of the dependencies and how to build it from source? Thanks a lot!

@ThorstenFalk
Copy link
Collaborator Author

README.md has been extended accordingly.

@lmb-freiburg lmb-freiburg locked as resolved and limited conversation to collaborators Feb 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants