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

name 'process_bbox_predictions' is not defined in inference.ipynb #1168

Open
saraswat40 opened this issue Jan 24, 2023 · 0 comments
Open

name 'process_bbox_predictions' is not defined in inference.ipynb #1168

saraswat40 opened this issue Jan 24, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@saraswat40
Copy link

Not sure if I missed something but if I try to run: https://github.com/airctic/icevision/blob/master/notebooks/inference.ipynb

I get this error:
NameError Traceback (most recent call last)
in
----> 1 export_batch_inferences_as_coco_annotations(
2 preds=preds,
3 img_files=img_files,
4 transforms=valid_tfms,
5 class_map=class_map,

/usr/local/lib/python3.8/dist-packages/icevision/data/convert_records_to_coco_style.py in export_batch_inferences_as_coco_annotations(preds, img_files, transforms, class_map, output_filepath, info, licenses)
127 # process_bbox_predictions happens inplace, thus no new variable
128 for p in preds:
--> 129 process_bbox_predictions(
130 p, PIL.Image.open(Path(p.pred.filepath)), transforms.tfms_list
131 )

NameError: name 'process_bbox_predictions' is not defined

There is no process_bbox_predictions anywhere in the repository or in convert_records_to_coco_style.py. OTOH inference.ipynb was updated in Aug 2022 and convert_records_to_coco_style.py in 2021. Was there an associated change which was never committed?

@saraswat40 saraswat40 added the bug Something isn't working label Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant