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

Increase the dtype limit to maintain consistency with the gpu. #2724

Merged
merged 2 commits into from
Apr 3, 2023

Conversation

momo609
Copy link
Collaborator

@momo609 momo609 commented Apr 1, 2023

Motivation

Increase the dtype limit to maintain consistency with the gpu.

Modification

Intercept float16 data type

@CLAassistant
Copy link

CLAassistant commented Apr 1, 2023

CLA assistant check
All committers have signed the CLA.

@momo609 momo609 force-pushed the roialign branch 2 times, most recently from 6227ce5 to a12e709 Compare April 1, 2023 03:44
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
@zhouzaida zhouzaida merged commit f946a93 into open-mmlab:master Apr 3, 2023
momo609 added a commit to momo609/mmcv that referenced this pull request Apr 4, 2023
…with the GPU (open-mmlab#2724)

* Increase the dtype limit to maintain consistency with the gpu.

* update error message

Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>

---------

Co-authored-by: momo609 <963372609.qq.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
CokeDong pushed a commit to CokeDong/mmcv that referenced this pull request Apr 24, 2023
…with the GPU (open-mmlab#2724)

* Increase the dtype limit to maintain consistency with the gpu.

* update error message

Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>

---------

Co-authored-by: momo609 <963372609.qq.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
MMCV 2.x
Awaiting triage
Development

Successfully merging this pull request may close these issues.

None yet

3 participants