-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Correct mmengine version requirement #2816
Conversation
Hi, thanks for your contribution. Please sign the CLA before we merge this PR. |
@zhouzaida I believe I have signed the CLA but the plugin is not reflecting that. I have clicked the recheck link several times already. |
Hi @wateryzephyr, We'd like to express our appreciation for your valuable contributions to the mmcv. Your efforts have significantly aided in enhancing the project's quality. If you're on WeChat, we'd also love for you to join our community there. Just add our assistant using the WeChat ID: openmmlabwx. When sending the friend request, remember to include the remark "mmsig + Github ID". Thanks again for your awesome contribution, and we're excited to have you as part of our community! |
@zhouzaida Please advise on how to proceed. |
Hi, it may be that the email address of the commit does not match your github profile. You can try to add it to your profile. |
This commit addresses the bug described in: open-mmlab#281
671d2cc
to
4f684b1
Compare
@zhouzaida good catch, I had signed in on a different account. It looks like the bot has edited their answer to now say that the CLA has been signed. |
This commit addresses the bug described in:
#2815
Motivation
Fix an incorrect version constraint on
mmengine
.Modification
Adjust the version constraint to one that is correct.
BC-breaking (Optional)
N/A
Use cases (Optional)
N/A
Checklist
Before PR:
After PR: