Skip to content

Conversation

@rasbt
Copy link
Owner

@rasbt rasbt commented Feb 4, 2024

No description provided.

@pep8speaks
Copy link

pep8speaks commented Feb 4, 2024

Hello @rasbt! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 24:80: E501 line too long (89 > 79 characters)
Line 38:80: E501 line too long (83 > 79 characters)
Line 55:80: E501 line too long (84 > 79 characters)
Line 61:80: E501 line too long (99 > 79 characters)
Line 66:80: E501 line too long (87 > 79 characters)
Line 68:80: E501 line too long (96 > 79 characters)
Line 78:80: E501 line too long (89 > 79 characters)
Line 79:71: W291 trailing whitespace
Line 83:80: E501 line too long (97 > 79 characters)
Line 88:80: E501 line too long (86 > 79 characters)
Line 89:80: E501 line too long (81 > 79 characters)
Line 90:80: E501 line too long (82 > 79 characters)
Line 101:62: W291 trailing whitespace
Line 133:55: W291 trailing whitespace
Line 159:38: W291 trailing whitespace
Line 197:80: E501 line too long (80 > 79 characters)
Line 225:34: W291 trailing whitespace

Comment last updated at 2024-02-04 16:06:06 UTC

@rasbt rasbt merged commit 18a02af into main Feb 4, 2024
Intelligence-Manifesto pushed a commit to Intelligence-Manifesto/LLMs-from-scratch that referenced this pull request Apr 2, 2024
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.

3 participants