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

Switch measurements to be loaded with dask rather than vaex #582

Draft
wants to merge 10 commits into
base: dev
Choose a base branch
from

Conversation

ddobie
Copy link
Collaborator

@ddobie ddobie commented Jan 17, 2025

No description provided.

@pep8speaks
Copy link

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

Line 1756:1: W293 blank line contains whitespace
Line 1802:1: W293 blank line contains whitespace
Line 1803:80: E501 line too long (101 > 79 characters)
Line 1804:1: W293 blank line contains whitespace
Line 1812:1: W293 blank line contains whitespace
Line 1816:1: W293 blank line contains whitespace
Line 1817:9: E265 block comment should start with '# '
Line 1818:1: W293 blank line contains whitespace
Line 1819:9: E265 block comment should start with '# '
Line 1820:9: E265 block comment should start with '# '
Line 1821:1: W293 blank line contains whitespace
Line 1824:80: E501 line too long (88 > 79 characters)
Line 1829:1: W293 blank line contains whitespace
Line 1830:9: E265 block comment should start with '# '
Line 1831:9: E265 block comment should start with '# '
Line 1832:1: W293 blank line contains whitespace
Line 1833:1: W293 blank line contains whitespace
Line 1834:9: E303 too many blank lines (2)
Line 1834:43: E226 missing whitespace around arithmetic operator
Line 1836:1: W293 blank line contains whitespace
Line 1837:9: E265 block comment should start with '# '

Line 86:80: E501 line too long (83 > 79 characters)
Line 112:80: E501 line too long (82 > 79 characters)
Line 227:17: E124 closing bracket does not match visual indentation
Line 234:17: E124 closing bracket does not match visual indentation
Line 406:40: W291 trailing whitespace
Line 430:1: W293 blank line contains whitespace
Line 431:9: E303 too many blank lines (2)
Line 1009:1: W293 blank line contains whitespace
Line 1035:1: W293 blank line contains whitespace
Line 1125:80: E501 line too long (94 > 79 characters)
Line 1140:9: E265 block comment should start with '# '
Line 1142:13: E265 block comment should start with '# '
Line 1143:13: E265 block comment should start with '# '
Line 1144:13: E265 block comment should start with '# '
Line 1145:13: E265 block comment should start with '# '
Line 1146:13: E265 block comment should start with '# '
Line 1147:13: E265 block comment should start with '# '
Line 1148:13: E265 block comment should start with '# '
Line 1149:23: E231 missing whitespace after ':'
Line 1150:24: E231 missing whitespace after ':'
Line 1154:13: E265 block comment should start with '# '
Line 1155:13: E265 block comment should start with '# '
Line 1156:27: E231 missing whitespace after ':'
Line 1171:1: W293 blank line contains whitespace
Line 1191:80: E501 line too long (85 > 79 characters)
Line 1192:1: W293 blank line contains whitespace
Line 1194:80: E501 line too long (85 > 79 characters)
Line 1195:1: W293 blank line contains whitespace
Line 1196:62: E226 missing whitespace around arithmetic operator
Line 1196:80: E501 line too long (84 > 79 characters)
Line 1351:1: W293 blank line contains whitespace
Line 1409:80: E501 line too long (80 > 79 characters)

Line 753:1: E302 expected 2 blank lines, found 1
Line 755:34: E252 missing whitespace around parameter equals
Line 755:35: E252 missing whitespace around parameter equals
Line 759:1: W293 blank line contains whitespace
Line 769:40: E226 missing whitespace around arithmetic operator
Line 769:50: E226 missing whitespace around arithmetic operator
Line 771:1: W293 blank line contains whitespace

@ddobie ddobie added the do not merge Do not merge yet label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants