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

Greater Wi-Fi card support, bug fixes, massive performance increases #104

Merged
merged 28 commits into from
Oct 18, 2024

Conversation

Batcastle
Copy link
Member

  • Add Realtek WiFi card support
  • When manually partitioning, if the user sets the boot flag on their EFI partition, do not try to reset
    • When resetting this flag, it actually UNSETS it. This can cause problems when booting on some motherboards
  • Could not install on SD cards recognized as /dev/mmcblk*
    • Partitioning formatting is the same as NVMe drives, added support there.
  • When not making or using a Home partition, when manually partitioning, if the user does not explicitly set this, Edamame didn't know what to do.
  • Fix typo in desktop launcher
  • Started to compile multiple files with Nuitka to improve performance

Batcastle and others added 15 commits February 7, 2024 00:04
Add support for Realtek cards. Also allows for failure of installation for main Restricted Extras packages, while drivers will still get installed.

Failures in both driver and main packages will not cause whole script to fail. But, the failure will still be reported.

Signed-off-by: Thomas Castleman <[email protected]>
Signed-off-by: Thomas Castleman <[email protected]>
build was adding development files to DEB file. Made sure they were removed.
This only affects Nuitka compilation.
 - When manually partitioning, if the user sets the boot flag on their EFI partition, do not try to reset
   - When resetting this flag, it actually UNSETS it. This can cause problems when booting on some motherboards
 - Could not install on SD cards recognized as /dev/mmcblk*
   - Partitioning formatting is the same as NVMe drives, added support there.
 - When not making or using a Home partition, if the user does not explicitly set this, Edamame didn't know what to do.
 - Specified Python3.11 in all files to ensure Nuitka works with the transition to 3.12 on Ubuntu 24.04
 - Removed block of code that was going unused in installer.py
   - Removing this block of code allowed to remove a dependency
@Batcastle Batcastle added bug Something isn't working enhancement New feature or request labels Mar 23, 2024
@Batcastle Batcastle self-assigned this Mar 23, 2024
@pep8speaks
Copy link

pep8speaks commented Mar 23, 2024

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

Line 43:31: E127 continuation line over-indented for visual indent
Line 50:101: E501 line too long (175 > 100 characters)
Line 51:101: E501 line too long (145 > 100 characters)
Line 63:31: E127 continuation line over-indented for visual indent
Line 66:101: E501 line too long (164 > 100 characters)

Line 30:1: E402 module level import not at top of file

Line 28:1: E402 module level import not at top of file
Line 29:1: E402 module level import not at top of file

Line 36:1: E402 module level import not at top of file
Line 38:1: E402 module level import not at top of file
Line 39:1: E402 module level import not at top of file
Line 286:5: E303 too many blank lines (2)
Line 465:101: E501 line too long (105 > 100 characters)
Line 472:101: E501 line too long (146 > 100 characters)
Line 987:101: E501 line too long (138 > 100 characters)
Line 1057:101: E501 line too long (107 > 100 characters)
Line 1070:101: E501 line too long (115 > 100 characters)
Line 1105:101: E501 line too long (103 > 100 characters)
Line 1453:31: E128 continuation line under-indented for visual indent
Line 1819:101: E501 line too long (102 > 100 characters)
Line 1821:9: E122 continuation line missing indentation or outdented
Line 1821:101: E501 line too long (107 > 100 characters)
Line 1822:101: E501 line too long (115 > 100 characters)
Line 1834:101: E501 line too long (114 > 100 characters)
Line 1835:101: E501 line too long (140 > 100 characters)

Line 30:1: E402 module level import not at top of file
Line 31:1: E402 module level import not at top of file
Line 111:5: E303 too many blank lines (2)

Line 37:1: E402 module level import not at top of file
Line 317:9: E722 do not use bare 'except'
Line 354:9: E722 do not use bare 'except'
Line 734:1: W391 blank line at end of file

Line 34:1: E402 module level import not at top of file

Line 67:13: E126 continuation line over-indented for hanging indent
Line 72:29: E126 continuation line over-indented for hanging indent
Line 77:29: E126 continuation line over-indented for hanging indent
Line 82:29: E126 continuation line over-indented for hanging indent
Line 83:25: E126 continuation line over-indented for hanging indent
Line 89:29: E126 continuation line over-indented for hanging indent
Line 94:29: E126 continuation line over-indented for hanging indent
Line 95:25: E126 continuation line over-indented for hanging indent
Line 99:25: E126 continuation line over-indented for hanging indent
Line 101:9: E126 continuation line over-indented for hanging indent
Line 120:39: E221 multiple spaces before operator
Line 138:1: E303 too many blank lines (4)
Line 158:5: E116 unexpected indentation (comment)
Line 197:22: E225 missing whitespace around operator
Line 298:5: E303 too many blank lines (2)
Line 411:101: E501 line too long (135 > 100 characters)
Line 414:101: E501 line too long (113 > 100 characters)
Line 418:101: E501 line too long (133 > 100 characters)
Line 421:101: E501 line too long (117 > 100 characters)
Line 423:101: E501 line too long (141 > 100 characters)
Line 426:101: E501 line too long (141 > 100 characters)
Line 439:17: E722 do not use bare 'except'
Line 519:101: E501 line too long (104 > 100 characters)

Line 63:5: E722 do not use bare 'except'

Line 35:9: E116 unexpected indentation (comment)
Line 36:9: E116 unexpected indentation (comment)
Line 37:13: E116 unexpected indentation (comment)
Line 76:1: E302 expected 2 blank lines, found 1

Line 51:22: E128 continuation line under-indented for visual indent
Line 59:22: E128 continuation line under-indented for visual indent
Line 61:1: E305 expected 2 blank lines after class or function definition, found 1
Line 171:30: E128 continuation line under-indented for visual indent

Line 183:13: E128 continuation line under-indented for visual indent
Line 255:59: W504 line break after binary operator

Line 30:1: E302 expected 2 blank lines, found 1

Line 62:101: E501 line too long (105 > 100 characters)
Line 81:1: E302 expected 2 blank lines, found 1
Line 84:101: E501 line too long (119 > 100 characters)
Line 98:101: E501 line too long (104 > 100 characters)
Line 112:16: E111 indentation is not a multiple of four
Line 113:16: E111 indentation is not a multiple of four
Line 138:48: E128 continuation line under-indented for visual indent
Line 161:101: E501 line too long (121 > 100 characters)
Line 174:101: E501 line too long (120 > 100 characters)
Line 175:101: E501 line too long (108 > 100 characters)

Line 32:1: E302 expected 2 blank lines, found 1
Line 79:1: E305 expected 2 blank lines after class or function definition, found 1

Line 149:40: E127 continuation line over-indented for visual indent
Line 150:40: E127 continuation line over-indented for visual indent
Line 151:40: E127 continuation line over-indented for visual indent
Line 155:36: E127 continuation line over-indented for visual indent
Line 156:36: E127 continuation line over-indented for visual indent
Line 157:36: E127 continuation line over-indented for visual indent
Line 205:31: E127 continuation line over-indented for visual indent
Line 224:55: E231 missing whitespace after ','
Line 291:31: E127 continuation line over-indented for visual indent
Line 295:28: E127 continuation line over-indented for visual indent
Line 299:35: E127 continuation line over-indented for visual indent
Line 305:27: E127 continuation line over-indented for visual indent
Line 324:31: E127 continuation line over-indented for visual indent
Line 353:101: E501 line too long (113 > 100 characters)
Line 365:101: E501 line too long (115 > 100 characters)
Line 380:27: E127 continuation line over-indented for visual indent
Line 388:13: E117 over-indented
Line 397:27: E127 continuation line over-indented for visual indent
Line 402:28: E127 continuation line over-indented for visual indent
Line 403:27: E127 continuation line over-indented for visual indent
Line 405:27: E127 continuation line over-indented for visual indent
Line 418:27: E127 continuation line over-indented for visual indent
Line 432:37: E127 continuation line over-indented for visual indent
Line 448:101: E501 line too long (127 > 100 characters)
Line 449:101: E501 line too long (109 > 100 characters)
Line 466:101: E501 line too long (131 > 100 characters)
Line 467:101: E501 line too long (119 > 100 characters)

Line 46:1: E303 too many blank lines (3)

Line 52:28: E127 continuation line over-indented for visual indent
Line 53:28: E127 continuation line over-indented for visual indent
Line 85:9: E265 block comment should start with '# '
Line 86:9: E265 block comment should start with '# '
Line 153:101: E501 line too long (118 > 100 characters)
Line 158:36: E128 continuation line under-indented for visual indent

Line 36:1: E402 module level import not at top of file
Line 38:1: E402 module level import not at top of file
Line 301:32: E128 continuation line under-indented for visual indent
Line 301:32: W503 line break before binary operator
Line 418:5: E303 too many blank lines (2)
Line 471:68: E231 missing whitespace after ','
Line 500:41: E127 continuation line over-indented for visual indent

Line 36:1: E402 module level import not at top of file
Line 400:101: E501 line too long (107 > 100 characters)
Line 413:101: E501 line too long (115 > 100 characters)
Line 415:43: E711 comparison to None should be 'if cond is not None:'
Line 448:101: E501 line too long (103 > 100 characters)
Line 472:5: E303 too many blank lines (2)
Line 571:1: E303 too many blank lines (3)

Line 30:1: E302 expected 2 blank lines, found 1
Line 33:101: E501 line too long (116 > 100 characters)

Line 38:29: E231 missing whitespace after ','
Line 44:57: W504 line break after binary operator
Line 58:58: E231 missing whitespace after ','
Line 61:58: E231 missing whitespace after ','

Comment last updated at 2024-07-22 01:51:01 UTC

Signed-off-by: Thomas Castleman <[email protected]>
This will hide the KDE panel at boot time. This is not complete, we need to disable desktop icons too. However, it's a good deal better than what we had.

This also allows the $PATH binary to be simpler, not hooking into the Python interpreter at all. We had to move all the Python code it was running into engine.py

Signed-off-by: Thomas Castleman <[email protected]>
Signed-off-by: Thomas Castleman <[email protected]>
Signed-off-by: Thomas Castleman <[email protected]>
Signed-off-by: Thomas Castleman <[email protected]>
Signed-off-by: Thomas Castleman <[email protected]>
Signed-off-by: Thomas Castleman <[email protected]>
@SenlisO
Copy link
Member

SenlisO commented Oct 18, 2024

Is this pull request still valid, or has it been overcome by events?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants