Skip to content

Feature/issue 146 call train config evaluation from train config generation app#147

Merged
daavoo merged 2 commits into
masterfrom
feature/issue-146-Call_train-config_evaluation_from_train-config_generation_app
May 6, 2021
Merged

Feature/issue 146 call train config evaluation from train config generation app#147
daavoo merged 2 commits into
masterfrom
feature/issue-146-Call_train-config_evaluation_from_train-config_generation_app

Conversation

@mmeendez8
Copy link
Copy Markdown
Contributor

Close #146

@mmeendez8 mmeendez8 requested review from daavoo and igonro April 19, 2021 16:30
Comment on lines +161 to +173
coco_ground_truth = load_ground_truth_file(ground_truth_file)

df_images, df_annotations = coco_ground_truth_to_dfs(coco_ground_truth)
df_images, df_annotations = coco_ground_truth_to_dfs(coco_ground_truth)

df_annotations = join_annotations_with_image_sizes(df_annotations, df_images)
df_annotations = join_annotations_with_image_sizes(df_annotations, df_images)

df_annotations = filter_zero_area_bboxes(df_annotations)
df_annotations = filter_zero_area_bboxes(df_annotations)

df_annotations = scale_bbox_dimensions(df_annotations, input_size=input_size)
df_annotations = scale_bbox_dimensions(
df_annotations, input_size=input_size, keep_ratio=keep_ratio
)

df_annotations = get_scale_and_ratio(df_annotations, prefix="scaled")
Copy link
Copy Markdown
Contributor Author

@mmeendez8 mmeendez8 Apr 19, 2021

Choose a reason for hiding this comment

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

These lines are repeated in train-config generation and train-config evaluation...
We might move this into a new function but not sure about where and if it is really necessary

@daavoo daavoo merged commit e878ba9 into master May 6, 2021
@daavoo daavoo deleted the feature/issue-146-Call_train-config_evaluation_from_train-config_generation_app branch May 6, 2021 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Call train-config evaluation from train-config generation app

3 participants