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

Support squeezed label as input in paddle.metric.Accuracy #28535

Merged
merged 4 commits into from
Nov 16, 2020

Conversation

qingqing01
Copy link
Contributor

@qingqing01 qingqing01 commented Nov 11, 2020

PR types

Bug fixes

PR changes

APIs

Describe

  • Support squeezed label in paddle.metric.Accuracy
  • Add unit testing
  • Update doc in cifar10

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot-old
Copy link

paddle-bot-old bot commented Nov 11, 2020

✅ This PR's description meets the template requirements!
Please wait for other CI results.

@@ -53,7 +53,9 @@ class Cifar10(Dataset):
:attr:`data_file` is not set. Default True

Returns:
Dataset: instance of cifar-10 dataset
Dataset: instance of cifar-10 dataset. If transform is None, the shape
of each data iterm is [3, 32, 32], and default dtype is float32.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

默认是返回pil的图片,shape应该是[32, 32, 3]

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code is:

图片

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LielinJiang Done. Thanks!

heavengate
heavengate previously approved these changes Nov 11, 2020
heavengate
heavengate previously approved these changes Nov 11, 2020
LielinJiang
LielinJiang previously approved these changes Nov 11, 2020
saxon-zh
saxon-zh previously approved these changes Nov 11, 2020
Copy link

@saxon-zh saxon-zh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Nov 12, 2020
@PaddlePaddle PaddlePaddle unlocked this conversation Nov 12, 2020
Copy link

@saxon-zh saxon-zh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

4 participants