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

建议更换您的teaser #18

Open
lxtGH opened this issue May 15, 2022 · 3 comments
Open

建议更换您的teaser #18

lxtGH opened this issue May 15, 2022 · 3 comments

Comments

@lxtGH
Copy link

lxtGH commented May 15, 2022

根据如下几个issue:

#16
#13

这篇论文的Teaser结果完完全全是错误的。

实时模型不能用多尺度crop测试结果却报告小图分辨率的速度。

全图测试结果attanet就76不到的水平。

另外速度测试没有用cuda的同步命令,速度测试也不对。

@lxtGH lxtGH changed the title 建议更换您的teaser,不然这个领域后续没法发展。。 建议更换您的teaser May 15, 2022
@Ilareina
Copy link

你好,由于这个账号我目前无法登陆,所以很多信息没办法更新上去,我先用新的账号回复一下。

  1. 当时放这个repo的时候,我们有在issue里回答说这个repo不是我们训练测试实际采用的文件,只是放出来给大家参考一下,我们在自己的代码中关于实时性精度测试的时候是没有用crop这些操作的;
  2. 关于速度测试这一点我们也有在issue里回答过,有没有cuda同步命令不影响我们模型的速度;
  3. 由于这个文章比较久了,训练的代码没有保存下来,但是保留了部分训练权重,我重新上传一份(密码1234),大家可以下载测试;
  4. 我们的模型在训练时精度浮动会比较大,这是模型的一个缺点,当时也是训练多次才得到最佳的精度;
  5. 之前的邮箱已经弃用了,如果有论文相关的问题可以通过这个新邮箱(qisong@link.cuhk.edu.cn)联系我,我会及时解答。

@lxtGH
Copy link
Author

lxtGH commented May 22, 2022

4,那你的model在哪呢??
#13 (comment)
另外不能1024 * 512 测速度,却报告全图结果。这是常识性错误。你的结果是真的1024 * 512 出来的嘛?

@Ilareina
Copy link

Ilareina commented May 22, 2022 via email

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

2 participants