Skip to content
This repository has been archived by the owner on Jul 2, 2021. It is now read-only.

fix _check_available in SBDInstanceSegmentationDataset #577

Merged
merged 2 commits into from
Apr 19, 2018

Conversation

Hakuyume
Copy link
Member

======================================================================
ERROR: test_sbd_instance_segmentation_dataset (test_sbd_instance_segmentation_dataset.transplant_class.<locals>.C)  parameter: {'split': 'train'}
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/chainercv/tests/datasets_tests/sbd_tests/test_sbd_instance_segmentation_dataset.py", line 19, in setUp
    self.dataset = SBDInstanceSegmentationDataset(split=self.split)
  File "/usr/local/lib/python3.5/dist-packages/chainercv-0.8.0-py3.5-linux-x86_64.egg/chainercv/datasets/sbd/sbd_instance_segmentation_dataset.py", line 45, in __init__
    _check_available()
  File "/usr/local/lib/python3.5/dist-packages/chainercv-0.8.0-py3.5-linux-x86_64.egg/chainercv/datasets/sbd/sbd_instance_segmentation_dataset.py", line 22, in _check_available
    'so the dataset cannot be loaded.'
TypeError: category must be a Warning subclass, not 'str'

@Hakuyume Hakuyume changed the title fix check_available in SBDInstanceSegmentationDataset fix _check_available in SBDInstanceSegmentationDataset Apr 19, 2018
@knorth55
Copy link
Contributor

Thank you so much !

@@ -21,7 +21,7 @@ def _check_available():
'SciPy is not installed in your environment,',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the comma at the end of this line causes this error.
I'm not sure, but RuntimeWarning does not matter according to https://github.com/chainer/chainercv/blob/master/chainercv/extensions/vis_report/detection_vis_report.py#L17-L22

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see. I updated.

@yuyu2172 yuyu2172 added the bug label Apr 19, 2018
@yuyu2172 yuyu2172 added this to the v0.9 milestone Apr 19, 2018
@yuyu2172 yuyu2172 merged commit c9b3b8e into chainer:master Apr 19, 2018
@Hakuyume Hakuyume deleted the fix-check-available-in-sbd branch April 19, 2018 08:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants