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

fix typos #3543

Merged
merged 71 commits into from Jul 19, 2023
Merged

fix typos #3543

merged 71 commits into from Jul 19, 2023

Conversation

RainRat
Copy link
Contributor

@RainRat RainRat commented Jul 4, 2023

No description provided.

RainRat added 30 commits May 4, 2023 14:24
@github-actions
Copy link

github-actions bot commented Jul 4, 2023

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

@andrewm4894 andrewm4894 enabled auto-merge (squash) July 12, 2023 15:24
@RainRat RainRat requested a review from jordiclive as a code owner July 19, 2023 04:27
Copy link
Collaborator

@andreaskoepf andreaskoepf left a comment

Choose a reason for hiding this comment

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

Thanks!

@@ -53,7 +53,7 @@ def test_dataset_entry_no_context(pythia_tokenizer):
# check if targets are as expected
assert pythia_tokenizer.decode(batch.targets[0]) == expected_decoded_targets[0]

# check if masking is correct. Note that we mask the system aswell
# check if masking is correct. Note that we mask the system as well
Copy link
Collaborator

Choose a reason for hiding this comment

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

From a swollen rump, to "too" 😂

Copy link
Collaborator

@bitplane bitplane left a comment

Choose a reason for hiding this comment

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

All looks good to me - very keen eye or a good spell/grammar checker. I had to actually check manually with some of them because eyeballing the differences didn't work. Nice work :)

@andrewm4894 andrewm4894 merged commit e421d1d into LAION-AI:main Jul 19, 2023
10 checks passed
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.

None yet

5 participants