cirKITers / mind-the-qapp

An Application for Visualizing the Crux in Quantum Machine Learning
MIT License
0 stars 0 forks source link

Cleanup & Refactoring #30

Closed stroblme closed 1 day ago

stroblme commented 2 months ago

This PR addresses resolves #7 by

pep8speaks commented 2 months ago

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

Line 104:80: E501 line too long (87 > 79 characters)

Line 50:80: E501 line too long (83 > 79 characters) Line 66:80: E501 line too long (85 > 79 characters) Line 84:80: E501 line too long (92 > 79 characters) Line 108:80: E501 line too long (88 > 79 characters) Line 124:80: E501 line too long (89 > 79 characters) Line 149:80: E501 line too long (94 > 79 characters)

Line 45:80: E501 line too long (89 > 79 characters) Line 55:80: E501 line too long (83 > 79 characters) Line 85:80: E501 line too long (85 > 79 characters) Line 143:80: E501 line too long (84 > 79 characters) Line 159:80: E501 line too long (85 > 79 characters) Line 191:80: E501 line too long (81 > 79 characters) Line 209:80: E501 line too long (88 > 79 characters)

Comment last updated at 2024-08-12 13:07:42 UTC
stroblme commented 2 months ago

most of the pep8 complains affect the html-style app structure. I would suggest to ignore as the code is well-readable and the few lines that have >88 chars fit well within the scheme, don't justify re-structuring.