-
Notifications
You must be signed in to change notification settings - Fork 229
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
[Bug] distill for yolox_s failed #184
Comments
This should not be a bug, the resize of yolox is in the forward, which will cause the teacher and student input picture size to be inconsistent. https://github.com/open-mmlab/mmdetection/blob/master/mmdet/models/detectors/yolox.py#L93 This issue #17 may help you. |
reference to your reply, I added: align_methods=[
dict(method='YOLOX._preprocess', import_module='mmdet.models')
], but it didn't take effect. There is a note in if align_methods is None:
self.context_manager = None
else:
# To obtain the python function's outputs, there will build a
# specific context manager. When enter the context manager, the
# functions will be rewrite. The context manager could record
# inputs or outputs of the functions , and pass from teachr to
# student. When exit the context manager, the rewritten functions
# will restore.
self.context_manager = ConversionContext(align_methods) but, in |
Same error as before adding align_method? |
Yes. The config after adding align_method is as follows:
|
I just submitted a pr and added an example about yolox. |
ths, solved |
hi, I have a new problem. This problem is not encountered when the iteration total epoch setting is small (<10). It is encountered when it is large (20, 50, etc.). Is it related to
|
I think this is caused by the YOLOXModeSwitchHook of the yolox model. You can rewrite this hook and add model.bbox_ head.use_ L1=True is changed to model.architecture.model.bbox_ head.use_ l1 = True |
* add docs about config comment * fix blank * fix comment * fix comment * fix comment * fix comment
* [enhancement] Reorganizing OpenMMLab projects in readme * add MMPose in supported codebase list * add MMPose in supported codebase list
Describe the bug
The original Backbone for yolox_s is CSPDarknet. I use ResNet18 instead. The detailed configuration is as follows:
When it runs to the epoch 2, the following error occurs:
Correct situation, no matter preds_S or preds_T, its shape should be (batch_size, 3, 80, 80), (batch_size, 3, 40, 40) or (batch_size, 3, 20, 20). But when the above error occurs, in my debugging, various sizes have appeared.
The text was updated successfully, but these errors were encountered: