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

Adding the EFDM method, CVPR2022 #36

Merged
merged 3 commits into from
Mar 17, 2022
Merged

Conversation

YBZh
Copy link
Contributor

@YBZh YBZh commented Mar 16, 2022

Thanks for your excellent codebase of Dassl.

Based on Dassl, I have validated a novel method, named EFDM, which has been accepted by the CVPR2022 (the paper can be found at: https://arxiv.org/abs/2203.07740)

In this pull request, I am adding the EFDM into Dassl to make it more powerful. Thanks for your time and consideration.

@KaiyangZhou
Copy link
Owner

Congrats! And glad to know that Dassl is helpful to your research! :)

I've checked the submitted files and they look fine.

So before merging the PR, I'd like you to run linter.sh to format the code and make sure there is no lingering warnings (let me know if you have done this).

Feel free to create a new issue titled "how to run EFDM (CVPR'22)" and provide instructions. Practitioners who are interested in using your work would want to know.

@YBZh
Copy link
Contributor Author

YBZh commented Mar 17, 2022

Thanks for your quick reply.

I have formatted the codes with the "linter.sh".

I will create a new issue titled "how to run EFDM (CVPR'22)" and provide a link to the repository: https://github.com/YBZh/EFDM, where detailed guidelines will be available.

Thanks again for your time.

@KaiyangZhou KaiyangZhou merged commit f343b3e into KaiyangZhou:master Mar 17, 2022
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