-
-
Notifications
You must be signed in to change notification settings - Fork 120
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
[Partially fixed] Wrong permissions for Klipper virtual env #134
Comments
Hey Frix! First of all, love Shake&Tune, you've done an awesome job with that, i will probably look into porting some of your work to the Realtime analysis tool once we get all the small stuff out of the way in 2.1. If you want to be involved in any way, don't hesitate to reach out! I actually considered preinstalling shake&tune but as much of our attention are currently going towards hybrid, we'd need to make some changes/additions, so it got put on the backburner. I might refer people to your docs though, they're hilarious and great at the same time :D Anyway! About the issue, not exactly sure when that happens as klipper is installed as pi, actually the CustomPiOS module comes straight from MainsailOS, i don't think i made any modifications. Might be a third party add-on. Currently looking into what's going on before just doing the "dumb thing" and chown'ing everything at the end. Will get back to you shortly, thanks for reaching out! |
@Frix-x Found the issue, will be fixed in 2.1-RC2. I have a small request, when linking klippy extensions check if
|
I will issue an update to 2.1 and 2.0 that recursively chowns the klippy-env. Will close this when done. |
@smwoodward i already explained to you why this doesn't do what you think it does. RatOS-configurator generates RatOS.cfg, and it runs as BASE_USER, same as moonraker (which is what you're interacting with when editing configs in mainsail). Making it not writable by BASE_USER, would break it. |
This comment was marked as abuse.
This comment was marked as abuse.
Hey! First of all, thanks for the kind words about Shake&Tune! I'm glad to hear that you’re considering porting some of the work to your realtime analysis tool. I'd love to be involved, so feel free to reach out whenever you need. As for future developments on my side, I am planning to add support for hybrid printers in Shake&Tune as well. While I'm still figuring out the specifics, some elements should be available soon. I'd be happy to collaborate on this if you have any input or requirements from your side as I don't have any hybrid printer on hand at the moment... Regarding the initial issue: thank you for addressing the problem so promptly! And I can certainly update Shake&Tune to register with the Ratos configurator if it exists (or if you want to do it by yourself by opening a PR, I'll be happy to merge it). However, I have a question: does this process support registering an entire folder? Shake&Tune consists of multiple files, so I want to ensure that the registration and management through Ratos will handle all of them appropriately. |
@Frix-x Awesome, will hit you up on discord when i get back to working on it :)
For sure! We're experimenting with a few things for measuring the Y's individually (or at least try to isolate them as much as possible) for doing belt tension graphs. I'll be happy to give it a go, would like to replace all of our graph scripts with shake tune in the future. I'm sure @HelgeKeck would want to play with it too.
Currently it only handles the extras (specifically to have recovery options in case the user ran "hard recovery" on klipper through moonraker). There are plans to work out a more full-fledged plugin system in the future, but that is quite a long way out still. As far as i know, most of the scripts run standalone, right? So as long as Shake&Tune is registered with the moonraker update manager, those should be safe. I can submit a PR if you want, just have to get a bunch of stuff off my list first :) |
And how he wants |
There is a lot that could be done with Shake&Tune and single toolhead and IDEX hybrid printers, cant wait to try it out |
What happened
Hi, I'm Shake&Tune dev and since I switched the tool to a Klipper plugin (v4), several people have told me that there is a problem installing it when also using RatOS:
After some troubleshooting, it's due to the Klipper venv not having the correct permissions: some of the packages are installed as root, and thus it's not possible to install/update any of them from a user script (like the Shake&Tune install script). I'm pretty sure this is unwanted and probably a bug.
What did you expect to happen
Install script of Shake&Tune activate the Klipper venv and install the required packages correctly without permissions errors
How to reproduce
Just try to run
wget -O - https://raw.githubusercontent.com/Frix-x/klippain-shaketune/main/install.sh | bash
on a RatOS installed printerAdditional information
Here are the relevant issues on my side:
Frix-x/klippain-shaketune#116
Frix-x/klippain-shaketune#127
The workaround to make it work currently is to issue manually:
And then Shake&Tune can be installed correctly
The text was updated successfully, but these errors were encountered: