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

[multimodal] Upgrade preset and update the finetuning example for object detection #3262

Merged
merged 31 commits into from
Jun 7, 2023

Conversation

FANGAreNotGnu
Copy link
Contributor

@FANGAreNotGnu FANGAreNotGnu commented Jun 2, 2023

For benchmarking result for three new presets, check internal quip document "Autogluon Object Detection Benchmarking in 0.8.0".

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@FANGAreNotGnu FANGAreNotGnu changed the title [wip][multimodal]Add dino preset [wip][multimodal] Upgrade detection's preset Jun 2, 2023
@FANGAreNotGnu FANGAreNotGnu changed the title [wip][multimodal] Upgrade detection's preset [wip][multimodal] Upgrade preset for object detection Jun 2, 2023
@FANGAreNotGnu FANGAreNotGnu changed the title [wip][multimodal] Upgrade preset for object detection [wip][multimodal] Upgrade preset and update the finetuning example for object detection Jun 3, 2023
@FANGAreNotGnu FANGAreNotGnu changed the title [wip][multimodal] Upgrade preset and update the finetuning example for object detection [WIP][multimodal] Upgrade preset and update the finetuning example for object detection Jun 3, 2023
@FANGAreNotGnu FANGAreNotGnu changed the title [WIP][multimodal] Upgrade preset and update the finetuning example for object detection [multimodal] Upgrade preset and update the finetuning example for object detection Jun 5, 2023
@github-actions
Copy link

github-actions bot commented Jun 5, 2023

Job PR-3262-cf0f781 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-3262/cf0f781/index.html

Copy link
Contributor

Choose a reason for hiding this comment

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

Why to change errors to warnings? Are there cases where class number or classes are missing but still work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because in test_preset.py we did not provide sample_data_path and new DINO model does not have a default CLASSES value and the test will fail.

if num_gpus is not None:
hyperparameters["env.num_gpus"] = num_gpus
if val_metric is not None:
hyperparameters["optimization.val_metric"] = val_metric
Copy link
Contributor

Choose a reason for hiding this comment

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

PR #3279 removes optimization.val_metric and adds validation_metric in init(). Better to merge #3279 first and rebase on it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can just remove the val_metric here. Later I'll test different val_metric's performance and add it back

@github-actions
Copy link

github-actions bot commented Jun 7, 2023

Job PR-3262-26f8ab2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-3262/26f8ab2/index.html

Copy link
Contributor

@zhiqiangdon zhiqiangdon left a comment

Choose a reason for hiding this comment

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

LGTM.

@FANGAreNotGnu FANGAreNotGnu merged commit 7a64631 into autogluon:master Jun 7, 2023
28 checks passed
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.

None yet

2 participants