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

Result of 'Ours TC Visible' in Table 2 #2

Open
xingchenzhang opened this issue Jan 7, 2021 · 12 comments
Open

Result of 'Ours TC Visible' in Table 2 #2

xingchenzhang opened this issue Jan 7, 2021 · 12 comments

Comments

@xingchenzhang
Copy link

Hi,

Thank you very much for this very interesting work!

I have a question about the production of 'Ours TC Visible' shown in Table 2. Are you using 'cfg/yolov3_kaist.cfg' and 'kaist_sisible_detector.weights' to produce the results? Could you please help to clarify?

Bests,
Xingchen

@indigohana
Copy link

indigohana commented Jan 9, 2021

Hi, I have the same question with u.But the cfg/yolov3_kaist.cfg doesn't have the 'condconv' module, so I am very confused about it.I guess 'Ours TC visible' was producted by TC res groups/TC all + ‘kaist_visible_detector.weigths'.

@mrkieumy
Copy link
Owner

Hi, Ours TC Visible (task-conditioned visible) used cfg/yolov3_kaist_tc_det.cfg. Any methods that have 'tc" name will use the cfg file with tc. Thanks!

@indigohana
Copy link

Hi, Ours TC Visible (task-conditioned visible) used cfg/yolov3_kaist_tc_det.cfg. Any methods that have 'tc" name will use the cfg file with tc. Thanks!

Hi,I have a question about the difference between 'Ours TC visible' and 'Ours TC Det'. Could you please help to clarify,thanks!

@xingchenzhang
Copy link
Author

Hi, Ours TC Visible (task-conditioned visible) used cfg/yolov3_kaist_tc_det.cfg. Any methods that have 'tc" name will use the cfg file with tc. Thanks!

Hi,

Thank you very much for your reply.

So you mean 'Ours TC Visible' is produced using cfg/yolov3_kaist_tc_det.cfg and 'kaist_visible_detector.weigths'? Did you train this model using only RGB images from KAIST?

Thanks a lot!

@xingchenzhang
Copy link
Author

Hi, Ours TC Visible (task-conditioned visible) used cfg/yolov3_kaist_tc_det.cfg. Any methods that have 'tc" name will use the cfg file with tc. Thanks!

Hi,I have a question about the difference between 'Ours TC visible' and 'Ours TC Det'. Could you please help to clarify,thanks!

Maybe the weights are different. I think they are trained using different traning sets.

@xingchenzhang
Copy link
Author

Hi, Ours TC Visible (task-conditioned visible) used cfg/yolov3_kaist_tc_det.cfg. Any methods that have 'tc" name will use the cfg file with tc. Thanks!

Hi,

When I tried to run with 'cfg/yolov3_kaist_tc_det.cfg' and 'kaist_visible_detector.weigts', I got the following error:
File "/home/xingchen/Documents/Imperial/Code/RGB2IR-GAN/pedestrian-detection/task-conditioned-master/utils.py", line 117, in get_all_boxes
tot = output[0]['x'].data.size(0)
KeyError: 'x'

But when I used 'cfg/yolov3_kaist.cfg' and 'kaist_visible_detector.weigts', it worked and gave reasonable results. However, you said you used 'cfg/yolov3_kaist_tc_det.cfg' . Have you encountered any problems?

@mrkieumy
Copy link
Owner

Hi, Ours TC Visible (task-conditioned visible) used cfg/yolov3_kaist_tc_det.cfg. Any methods that have 'tc" name will use the cfg file with tc. Thanks!

Hi,I have a question about the difference between 'Ours TC visible' and 'Ours TC Det'. Could you please help to clarify,thanks!

Hi, Ours TC visible is to train on visible-only images of KAIST dataset with task-conditioned network (starting from yolov3.weight), while Ours TC Det is to train on thermal images of KAIST dataset with task-conditioined network (starting from visible weight from previous one).

@mrkieumy
Copy link
Owner

Hi, Ours TC Visible (task-conditioned visible) used cfg/yolov3_kaist_tc_det.cfg. Any methods that have 'tc" name will use the cfg file with tc. Thanks!

Hi,

When I tried to run with 'cfg/yolov3_kaist_tc_det.cfg' and 'kaist_visible_detector.weigts', I got the following error:
File "/home/xingchen/Documents/Imperial/Code/RGB2IR-GAN/pedestrian-detection/task-conditioned-master/utils.py", line 117, in get_all_boxes
tot = output[0]['x'].data.size(0)
KeyError: 'x'

But when I used 'cfg/yolov3_kaist.cfg' and 'kaist_visible_detector.weigts', it worked and gave reasonable results. However, you said you used 'cfg/yolov3_kaist_tc_det.cfg' . Have you encountered any problems?

Hi the problem because of the wrong return parameter between models (normal model and task-conditioned (TC) model). You can not load the normal weight (without tc) to the tc.cfg, you should load the tc.weight to corresponding tc.cfg because with the TC model, the output return of the network is different from normal model, so we can not use output[0]['x'].data.size(0). The easiest way to mitigate this problem is that you should load the corresponding *.cfg and *.weight. Good luck!

@indigohana
Copy link

Hi, Ours TC Visible (task-conditioned visible) used cfg/yolov3_kaist_tc_det.cfg. Any methods that have 'tc" name will use the cfg file with tc. Thanks!

Hi,
When I tried to run with 'cfg/yolov3_kaist_tc_det.cfg' and 'kaist_visible_detector.weigts', I got the following error:
File "/home/xingchen/Documents/Imperial/Code/RGB2IR-GAN/pedestrian-detection/task-conditioned-master/utils.py", line 117, in get_all_boxes
tot = output[0]['x'].data.size(0)
KeyError: 'x'
But when I used 'cfg/yolov3_kaist.cfg' and 'kaist_visible_detector.weigts', it worked and gave reasonable results. However, you said you used 'cfg/yolov3_kaist_tc_det.cfg' . Have you encountered any problems?

Hi the problem because of the wrong return parameter between models (normal model and task-conditioned (TC) model). You can not load the normal weight (without tc) to the tc.cfg, you should load the tc.weight to corresponding tc.cfg because with the TC model, the output return of the network is different from normal model, so we can not use output[0]['x'].data.size(0). The easiest way to mitigate this problem is that you should load the corresponding *.cfg and *.weight. Good luck!

Hello, thank you for your reply. I had the same problem when I trained with the FLIR dataset. I thought that cfg and model should correspond, did you have the same problem, or could you give me some advice? Thank you!

@gongyan1
Copy link

When I download the source code directly, use cfg / yolov3_ kaist_ tc_ det.cfg corresponding with weights/yolov3_ tc_ det_ thermal. model can run in the Evaluation.py, but with weights/yolov3_ tc_ det_ thermal.weights cannot be run. I am very confused about this error!
The error is:
tot = output[0]['x'].data.size(0)
KeyError: 'x'

@medicineqian
Copy link

When I download the source code directly, use cfg / yolov3_ kaist_ tc_ det.cfg corresponding with weights/yolov3_ tc_ det_ thermal. model can run in the Evaluation.py, but with weights/yolov3_ tc_ det_ thermal.weights cannot be run. I am very confused about this error! The error is: tot = output[0]['x'].data.size(0) KeyError: 'x'

Can you publish the results of your tests using cfg / yolov3_ kaist_ tc_ det.cfg and weights/yolov3_ tc_ det_ thermal? The results of my test are way off from the paper.
image
image

@gongyan1
Copy link

When I download the source code directly, use cfg / yolov3_ kaist_ tc_ det.cfg corresponding with weights/yolov3_ tc_ det_ thermal. model can run in the Evaluation.py, but with weights/yolov3_ tc_ det_ thermal.weights cannot be run. I am very confused about this error! The error is: tot = output[0]['x'].data.size(0) KeyError: 'x'

Can you publish the results of your tests using cfg / yolov3_ kaist_ tc_ det.cfg and weights/yolov3_ tc_ det_ thermal? The results of my test are way off from the paper. image image

Hello,I don't have pretrained weights. However, I once modified a version of this code. I'm not sure if that helps you. If you want a reference. please provide your email for me.

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

No branches or pull requests

5 participants