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

Eliminate some errors on bookworm #6

Merged
merged 4 commits into from
Dec 21, 2024
Merged

Conversation

spoelstraethan
Copy link
Contributor

No description provided.

@BLeeEZ
Copy link
Owner

BLeeEZ commented Dec 2, 2023

Great work. Thank you for the updates.

@BLeeEZ BLeeEZ closed this Dec 2, 2023
@BLeeEZ BLeeEZ reopened this Dec 2, 2023
@spoelstraethan
Copy link
Contributor Author

I think one change that might be useful in a future update would be to not enable the GPIO control of an external text display unless the user opts in. My first attempt at using this package was on a Pi400 with a Hyperpixel LCD attached and the GPIO changes were causing the screen to cycle through weird color tinting.

One thing I didn't notice in the README was how to pair a new device, I think in one attempt I paired using the UI Bluetooth settings and then restarted rpi-kvm and it was able to utilize that pairing. I think I later realized that when the service starts it sets up the Bluetooth in discoverable mode all the time? It would be nice to be able to turn that on and off (via the CLI or web) as you may not want the extra RF noise or people attempting to pair unexpectedly.

I also discovered that while you can easily connect and disconnect and select an active device via the web there's not a way to unselect a device to direct inputs to the host OS, disconnecting the currently active device is effectively the same thing but not intuitive.

I'll probably open some additional issues with these feature requests and if I get a chance poke it some of the implementation myself.

@BLeeEZ
Copy link
Owner

BLeeEZ commented Dec 2, 2023

These ideas all sound great. Every support is appreciated.

@argenkiwi
Copy link

argenkiwi commented Dec 18, 2024

Nice, I noticed the missing dependencies when trying to run the project on bookworm this morning.

Edit: I've just made a post mentioning this project. Have you considered enabling discussions for this repo? I hope you don't mind this kind of comments in the meantime.

@BLeeEZ
Copy link
Owner

BLeeEZ commented Dec 18, 2024

@spoelstraethan this PR is still marked as draft. I will submit this PR as soon as it is regularly open.

@argenkiwi I activated discussions for this project. Your post describes the reason why I started this project. I use a Dactyl Manuform with Colemak Mod-DH and wanted to have an easy way to access my devices.

@argenkiwi
Copy link

Thanks @BLeeEZ for enabling discussions! I hope my post will attract some attention to this project. I admit I found it by coincidence while looking into an issue for a similar project that I failed to get to work. Your project is so much tidier and easier to install!

@spoelstraethan spoelstraethan marked this pull request as ready for review December 18, 2024 21:36
@spoelstraethan
Copy link
Contributor Author

I haven't had a chance to test this recently, but if somebody can confirm it also works for them I think it is good to merge.

@BLeeEZ BLeeEZ merged commit 93c25ba into BLeeEZ:main Dec 21, 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