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

Customizable for different Installations #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Buadrabas
Copy link

I changed a few files and created a changes.md to add customizing of the installationspath

I changed a few files and created a changes.md to add customizing of
the installationspath
Copy link

@r14n r14n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to manually change the path to the klippy socket for my Ender 3 S1 to /home/pi/printer_data/comms/klippy.sock, so this change seems like it would be useful

@rosenhauer
Copy link

rosenhauer commented Feb 8, 2024

Like @r14n had to do I needed to change the socket path.

Here is a writeup of what I had to do.

In the “Check if Klipper's Application Programmer Interface (API) is enabled” section of the ReadMe
i. Mainsail already has the -a option but with a different path to the socket.
ii. Change the klipper -a path in the display system

  1. Orig is “-a /tmp/klippy_uds”
  2. Look in the klipper.log from mainsail for the actual socket being used
    a. In the log search for “Starting Klippy...” and the next lines will show the arguments
    b. Copy the path after the ‘-a’, argument.
    c. Should be “/home/pi/printer_data/comms/klippy.sock”
  3. Edit the printerInterface.py file
    a. Search for “self.ks = KlippySocket(“
    b. Change to match the log

Also note that the display CANNOT have firmware 3.1.9 or 3.2.0 they are incompatible. I had to go back to 3.0.4

https://www.creality.com/pages/download-ender-3-s1

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