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

fix AT_CHECK #57

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

sailfish009
Copy link

AT_CHECK should be replaced with TORCH_CHECK.

@joe-siyuan-qiao
Copy link
Owner

@sailfish009 Thanks for opening this PR. However, we do not plan to keep this repo updated as we have migrated it to the official mmdetection repo, which will always be updated with the newest environment, packages, and toolkits. Please open PR over there if you think it is relevant. Thanks!

@sailfish009
Copy link
Author

sailfish009 commented Jul 29, 2020

@joe-siyuan-qiao hi, i would love to use, but latest DetectoRS version seems to be unstable, at this moment, couldn't get same training result.

so i am using back-ported version of old DetectoRS.

--
Edit: now i am using latest mmdetection to use DetectoRS, and found the reason why it was unstable:
open-mmlab/mmdetection#3465

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.

2 participants