fix a bug when using base name of image_name to access annotation during recording confidence. #190
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Line 116 of autodistill/detection/detection_base_model.py file
Here the
image_names
should be full path instead of base name, because the keys inannotations
andimages
in dataset are full path.Line 51 of autodistill/detection/detection_base_model.py file
Here
os.path.splittext
should work with base name instead of full path due to my previous fix.Original code would have bug when you try to run
line 50
of this file. Theimage_name
was base name of the file instead of full path, but the keys forannotations
are full path.This bug won't show up until you set
record_confidence = True
for thelabel
method ofDetectionBaseModel
class.