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

[Minor] Fix typos #332

Merged
merged 3 commits into from
Jul 7, 2023
Merged

[Minor] Fix typos #332

merged 3 commits into from
Jul 7, 2023

Conversation

cpprhtn
Copy link
Contributor

@cpprhtn cpprhtn commented Jul 7, 2023

bachnorm-> batchnorm

imagenet -> ImageNet

Because ImageNet is a proper noun, it retains the ImageNet word

@netlify
Copy link

netlify bot commented Jul 7, 2023

Deploy Preview for pytorch-hub-preview ready!

Name Link
🔨 Latest commit 27a2f4c
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-hub-preview/deploys/64a7e038341ff90008b16b3f
😎 Deploy Preview https://deploy-preview-332--pytorch-hub-preview.netlify.app/ecosystem
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cpprhtn
Copy link
Contributor Author

cpprhtn commented Jul 7, 2023

#333
I'm trying to solve the CI error

@cpprhtn
Copy link
Contributor Author

cpprhtn commented Jul 7, 2023

@vmoens can you review about this?

@vmoens vmoens changed the title fix: typo err [Minor] Fix typo Jul 7, 2023
@vmoens vmoens changed the title [Minor] Fix typo [Minor] Fix typos Jul 7, 2023
@@ -12,7 +12,8 @@ conda install -y regex pillow tqdm boto3 requests numpy h5py scipy matplotlib un
conda install -y -c conda-forge librosa inflect

pip install -q fastBPE sacremoses sentencepiece subword_nmt editdistance
pip install -q visdom mistune filelock tokenizers packaging pandas
pip install -q visdom filelock tokenizers packaging pandas
pip install -q mistune==2.0.2
Copy link
Contributor

Choose a reason for hiding this comment

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

why is this needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know if it's a short thought
Checked the following logs and ran a downgrade.
https://circleci.com/gh/pytorch/hub/2694

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There was no problem with the history of merged without errors until 2022, and the mistune version at that time was 2.0.4.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

A build error was displayed.
What do you think about modifying all libraries installed before a specific date to a version

  • If the torch version goes down to the 1.x version, does it cause problems with interaction with other org pages?

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for the typo fixes @cpprhtn. Do you mind reverting those install changes in this PR so that we can merge it? We can follow-up on the install issue on another PR. I see you already opened #333, thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your review!

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Thanks @cpprhtn !

@NicolasHug NicolasHug merged commit 54269a4 into pytorch:master Jul 7, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants