search

ChenJinBIT / HPDA

7 stars 0 forks source link

代码无法复现出论文汇报的结果 #1

Open wyyfighting opened 3 years ago

wyyfighting commented 3 years ago

发现一个Bug，HPDA/models/cycle_gan_multiD_st_concat_pseudo_causal_model.py line 313，temporal feature和spatial feature位置反了。 修复之后依然无法复现出结果，恳请作者check训练代码无误并放出训练log及预训练模型。

ChenJinBIT commented 3 years ago

发现一个Bug，HPDA/models/cycle_gan_multiD_st_concat_pseudo_causal_model.py line 313，temporal feature和spatial feature位置反了。 修复之后依然无法复现出结果，恳请作者check训练代码无误并放出训练log及预训练模型。

谢谢您的建议，我们会再仔细检查代码。在8日邮件回复了关于训练的问题，并抄送了日志，但是您的邮箱zy1906917@buaa.edu.cn，提示投递失败，后又采用qq邮箱重新发了一份，如果仍然没有收到，可以再给我留一个邮箱，或者其他联系方式。

wyyfighting commented 3 years ago

感谢您的回复，我确实没有收到您的回复邮件，您可以把训练log和预训练模型等可以提供的材料发送到这个邮箱，谢谢！ 所以cycle_gan_multiD_st_concat_pseudo_causal_model.py line 313 是不是应该为先rec_B_t, 再rec_B_s？分别和flow与rgb对应。

在 2021-08-10 23:10:33，"ChenJinBIT" @.***> 写道：

temporal feature和spatial @.***，提示投递失败，后又采用qq邮箱重新发了一份，如果仍然没有收到，可以再给我留一个邮箱，或者其他联系方式。

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

wyyfighting commented 3 years ago

@.***，谢谢！ 所以cycle_gan_multiD_st_concat_pseudo_causal_model.py line 313 是不是应该为先rec_B_t, 再rec_B_s？分别和flow与rgb对应。

在 2021-08-10 23:10:33，"ChenJinBIT" @.***> 写道：

temporal feature和spatial @.***，提示投递失败，后又采用qq邮箱重新发了一份，如果仍然没有收到，可以再给我留一个邮箱，或者其他联系方式。

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

ChenJinBIT commented 3 years ago

@. @.，谢谢！ 所以cycle_gan_multiD_st_concat_pseudo_causal_model.py line 313 是不是应该为先rec_B_t, 再rec_B_s？分别和flow与rgb对应。 在 2021-08-10 23:10:33，"ChenJinBIT" @.> 写道： temporal feature和spatial @.，提示投递失败，后又采用qq邮箱重新发了一份，如果仍然没有收到，可以再给我留一个邮箱，或者其他联系方式。 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

核对之后发现，313行是写反了，按照改正后的代码和之前相同的参数配置，初步结果仍能超过其他对比方法。通过调参，应该可以进一步提升性能。您这里留的邮箱@.***，都是打了米字号的，想问您要个正确的邮箱，给您发之前的训练日志、训练好的模型等。