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

[CodeCamp2023-489]Add new configuration files for htc in mmdetection #10721

Open
wants to merge 8 commits into
base: dev-3.x
Choose a base branch
from

Conversation

Eiviento
Copy link

@Eiviento Eiviento commented Jul 29, 2023

Modify and provide a new configs————mmdet/configs/htc

Motivation

Give HTC all the configuration Add configuration and verify inference and training correctness

##Othres

I have verified the configuration file of htc。
I tried many times but the new configuration file modified by htc_x101-64x4d-dconv-c3-c5_fpn_ms-400-1400-16xb1-20e_coco.py failed to be verified.
The error cause is: ConvTranspose2d.init() got an unexpected keyword argument 'deform_groups'.
I'm not sure if it has anything to do with the original configuration parameter Settings.
I would appreciate it if you could give me some advice

@Eiviento Eiviento changed the title Eiviento/mmdet new config for htc [CodeCamp2023-489]Add new configuration files for htc in mmdetection Jul 29, 2023
@CLAassistant
Copy link

CLAassistant commented Jul 29, 2023

CLA assistant check
All committers have signed the CLA.

@OpenMMLab-Assistant-004
Copy link

Hi @Eiviento,

We'd like to express our appreciation for your valuable contributions to the mmdetection. Your efforts have significantly aided in enhancing the project's quality.
It is our pleasure to invite you to join our community thorugh Discord_Special Interest Group (SIG) channel. This is a great place to share your experiences, discuss ideas, and connect with other like-minded people. To become a part of the SIG channel, send a message to the moderator, OpenMMLab, briefly introduce yourself and mention your open-source contributions in the #introductions channel. Our team will gladly facilitate your entry. We eagerly await your presence. Please follow this link to join us: ​https://discord.gg/UjgXkPWNqA.

If you're on WeChat, we'd also love for you to join our community there. Just add our assistant using the WeChat ID: openmmlabwx. When sending the friend request, remember to include the remark "mmsig + Github ID".

Thanks again for your awesome contribution, and we're excited to have you as part of our community!

@hhaAndroid hhaAndroid changed the base branch from 3.x to dev-3.x August 8, 2023 01:48
hhaAndroid
hhaAndroid previously approved these changes Aug 8, 2023
@hhaAndroid
Copy link
Collaborator

@Eiviento Did you possibly locate the DCN code in the wrong place? Could you please provide the import path you are using?

@Eiviento
Copy link
Author

Eiviento commented Aug 8, 2023

@Eiviento Did you possibly locate the DCN code in the wrong place? Could you please provide the import path you are using?

from mmcv.cnn.bricks.wrappers import ConvTranspose2d as DCN

The changed IP address of the configuration file:https://github.com/Eiviento/mmdetection/blob/Eiviento/mmdet_new_config_htc/mmdet/configs/htc/htc_x101-64x4d-dconv-c3-c5_fpn_ms-400-1400-16xb1-20e_coco.py

@hhaAndroid
Copy link
Collaborator

@Eiviento
Your pull request has an issue. You shouldn't be deleting files, and in fact, DCN stands for DeformConv2dPack.

@MODELS.register_module('DCN')
--
class DeformConv2dPack(DeformConv2d):

@hhaAndroid
Copy link
Collaborator

@Eiviento Hi. Do you have any progress?

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

5 participants