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

[feat] add onnx-window model to backend (ppliteseg-nysbc-ccameron) #331

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

patrickcleeve2
Copy link
Contributor

DRAFT

@pep8speaks
Copy link

pep8speaks commented Jul 25, 2024

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

Line 205:80: E501 line too long (101 > 79 characters)
Line 212:80: E501 line too long (89 > 79 characters)

Line 108:80: E501 line too long (86 > 79 characters)
Line 110:1: E266 too many leading '#' for block comment
Line 111:1: E302 expected 2 blank lines, found 1
Line 132:1: E302 expected 2 blank lines, found 1
Line 156:80: E501 line too long (85 > 79 characters)
Line 163:1: W293 blank line contains whitespace
Line 171:80: E501 line too long (109 > 79 characters)
Line 172:1: W293 blank line contains whitespace
Line 173:80: E501 line too long (80 > 79 characters)
Line 179:12: E262 inline comment should start with '# '
Line 186:1: W293 blank line contains whitespace
Line 189:9: E266 too many leading '#' for block comment
Line 212:31: W291 trailing whitespace
Line 214:9: E122 continuation line missing indentation or outdented
Line 217:80: E501 line too long (105 > 79 characters)
Line 220:1: W293 blank line contains whitespace
Line 221:28: E231 missing whitespace after ','
Line 221:80: E501 line too long (125 > 79 characters)
Line 228:80: E501 line too long (86 > 79 characters)
Line 233:5: E122 continuation line missing indentation or outdented
Line 233:9: E231 missing whitespace after ','
Line 253:1: W293 blank line contains whitespace
Line 262:1: W293 blank line contains whitespace
Line 267:80: E501 line too long (87 > 79 characters)
Line 271:80: E501 line too long (93 > 79 characters)
Line 273:80: E501 line too long (94 > 79 characters)
Line 281:27: E203 whitespace before ':'
Line 282:27: E203 whitespace before ':'
Line 289:27: E203 whitespace before ':'
Line 290:27: E203 whitespace before ':'
Line 294:1: W293 blank line contains whitespace
Line 295:9: E303 too many blank lines (2)
Line 304:1: E302 expected 2 blank lines, found 1
Line 317:80: E501 line too long (109 > 79 characters)
Line 321:9: E266 too many leading '#' for block comment
Line 349:80: E501 line too long (105 > 79 characters)
Line 377:40: W291 trailing whitespace

Line 44:80: E501 line too long (86 > 79 characters)
Line 45:80: E501 line too long (83 > 79 characters)
Line 46:80: E501 line too long (82 > 79 characters)
Line 82:80: E501 line too long (86 > 79 characters)
Line 93:80: E501 line too long (85 > 79 characters)
Line 140:80: E501 line too long (87 > 79 characters)
Line 182:80: E501 line too long (88 > 79 characters)

Comment last updated at 2024-08-02 15:48:25 UTC

@patrickcleeve2 patrickcleeve2 force-pushed the feat-onnx-sliding-window branch from e375591 to 77365fd Compare July 26, 2024 04:23
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