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

evaluator fix #20

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

evaluator fix #20

wants to merge 3 commits into from

Conversation

Zanna-93
Copy link

@Zanna-93 Zanna-93 commented Feb 7, 2025

Description

  1. Inside the Evaluator class constructor a KeyError Exception was raised when instantiating gt array because keys inside ground_thruth dictionary were absolute paths of images and the access was attempted using only the filename retrieved with key.split("/")[-1] (evaluator.py:59)
  2. Restored previous allocation of merged_data dictionary because anAttributeError exception was raised when instruction gts = [list(item) for item in gt.xyxy] was executed, this because the gt object does not have an attribute named xyxy. That attribute exists in the predictions object

Comments left for in place explanations.

Type of change

Please delete options that are not relevant.

  • [X ] Bug fix (non-breaking change which fixes an issue)

How has this change been tested, please provide a testcase or example of how you tested the change?

Execution of roboflow_example.py script against a YOLOv5 PyTorch (txt) dataset.

Docs

  • Docs updated? Description of model_type CLI parameter and link to example

@CLAassistant
Copy link

CLAassistant commented Feb 7, 2025

CLA assistant check
All committers have signed the CLA.

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

Successfully merging this pull request may close these issues.

2 participants