-
Notifications
You must be signed in to change notification settings - Fork 15
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
Set settings over serial interface and save them #57
Comments
The app is written in Python using the Textualize TUI (Text-based user interface/Terminal user interface) library and I written it specifically for Windows(also Windows Terminal is recommended since CMD is old and the UI looks broken while the app still functions normally). The app is not that easy to distribute (you have to install python, make a virtual environment, install dependencies, run the app), this can be a limiting factor for not-so-technical users (but I guess most of us are technical since we talk about a DIY mouse). The installing process can also be further automated. I'm waiting for some feedback from users that use the Spacemouse more since I don't own one myself (yet), but I still have access to one occasionally. |
I forked your repo and implemented the changes more cleaner than in my repo (see: raulpetru@2a6cab5). It's not the best implementation. Also I might have disabled serial debugging since I commented the initial I want to write a function on Arduino that returns through serial the variables that we want to be included in the UI, then my app will pick this and build the UI. This way I don't have to hardcode the variables in my app, the app will request the variables from Spacemouse and will add them in the UI. This way we can use the same app with different firmwares. |
Great! So we can have a look at your approach!
We need to find a way to enable both use cases. Now, the input is waiting for only 2 ms and is trying to parse the input as a number. I suggest we add a little input queue, which evaluates the input, when it finds a line feed (new line key \n"). Then: If just a number is entered, we have the actual debug behavior. If e.g. "set transx 15 \n" ist transmitted the sensitivtiy for translation X is changed. (I suggest that we find shorter variable names...)
Yes, this is very good suggestion! |
I'm back with a new application, take a look when you find time (DIY SpaceMouse Profiles). I can do later a fork (maybe this week or the next one) of your firmware and apply the changes to make it compatible with the UI application. I haven't documented the process, but if you want to do it yourself you can compare the firmware from my repo with the original from @ChromeBee to see the changes. Edit: For some reason all one-file executables made with Pyinstaller give a false positive and are flagged as malware on Windows. I will commit some changes today to package the app in a directory instead of a single file to avoid false positives (fixed it in v1.0.1) |
Great to hear from you again! At the moment I don't have time to code it myself, but I'm looking forward to review and integrate you suggestions! |
@raulpetru created a fork with a python programm to send the sensitivity values over the serial interface. See
https://github.com/raulpetru/SpacemouseProfiles
With this feature request we want the following features:
The text was updated successfully, but these errors were encountered: