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

Incorrect reading of MNIST Images #8614

Open
benvcutilli opened this issue Jun 18, 2021 · 1 comment
Open

Incorrect reading of MNIST Images #8614

benvcutilli opened this issue Jun 18, 2021 · 1 comment
Labels
issue-checked stale Not updated for a longer period of time.

Comments

@benvcutilli
Copy link

This may be considered a minor quibble by most, but I do want to be accurate when using the MNIST dataset.

The documentation for chainer.datasets.get_mnist(...) implies that you do something like derive the 0 to scale values by just dividing the raw pixel value by 255. However, from the MNIST site (under "FILE FORMATS FOR THE MNIST DATABASE):

Pixels are organized row-wise. Pixel values are 0 to 255. 0 means background (white), 255 means foreground (black).

The same goes for the testing images. So, I believe you should be inverting your values output by get_mnist(...). Actually it would be best to give the option to invert them seeing as one may need to compare their results to other research that used the wrong method. I know there are workaround(s) for this, but this is technically a bug.

@stale
Copy link

stale bot commented Jan 8, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 30 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Not updated for a longer period of time. label Jan 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue-checked stale Not updated for a longer period of time.
Projects
None yet
Development

No branches or pull requests

2 participants