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
[OS Request] OctoPi-Klipper #330
Comments
I don't think you can do this. Version 1.6 of RPi-Imager will crash when you select this menu... |
@ghollingworth It seems to work for me on 1.6.2, and the one that introduced it was @foosel so I assume it works for her too. |
Did you test it on all platforms? |
I tested on the platform I have immediate access to (Windows) and assumed since it worked there I thought it was fine and supported. Can roll it back for now to not cause crashes, if that is indeed happening on other platforms, but for cases like this it would be good if there was some kind of JSON schema or some other form of dependable spec for the supported JSON structure, since apparently it is not self documenting and recursive (as in, looking at the master document does not hint at what works in sub trees). edit now also tested under Linux and couldn't repro any crashes, selected through all OctoPi submenus. What am I supposed to see, when should it crash? edit 2 asked a buddy with access to a Mac and he can't repro either. Everything on RPi Imager 1.6.2. |
It is fine since "subitems_url" is used. |
Thank you for clarifying @maxnet! |
Ah ok, So we'll remove the upper level Klipper entry |
Please don't confuse Klipper with Octo-Klipper 😅 Octo-Klipper did never made it into your main json file last I checked, which is why I put it into the OctoPi one to help out Guy. Octo-Klipper is OctoPrint with Klipper, but I think there might also be a Klipper-only image, possibly with other frontend alternatives on board. |
Well, everything is now in rpi-image if you check (i.e. as of the last hour or so...) Gordon |
All of these are not in the v2 json file, but if they were meant to be, I'll add them now. |
Give it a go now. I've added the OctoPi-Klipper entry as well, so it can be removed from OctoPi's json. I can understand the good intentions, but please don't use your own json files to circumvent the right process. |
Good point, the v2 file cannot have the sub-menu. Those will have to wait for a couple of days until we release a new rpi-imager update |
It's now merged into RPi Imager. See raspberrypi/rpi-imager#330
Can confirm. And the submenu-url from OctoPi's json should be gone too, see the above commit.
Noted. It was merely an attempt at helping out, no harm intended :) |
@ghollingworth Suggestion: If you guys have some kind of documentation, or even open the wiki here so we could document what should work, it would be spec-tacular :) @XECDesign We have no intention to go against due process. After the comment that this can actually break something @foosel removed the klipper entry. And after @maxnet 's comment that it should be fine it was added back again. We don't enjoy doing multiple work as do you. I do think though that rpi-imager has lots of potential to make people flash and boot usage-specific stuff on their pis. And that its worth figuring out how it all comes together. |
Quick correction to that, I didn't remove things in between these two time points as I was still assessing the situation and the "all good" from @maxnet came before I did the deed. But now the OctoPi-Klipper subentry is gone, the nightlies however are still in there (as a subitem url, not as subitems). |
Hey, so sorry to bug, but we need to update. Due to OctoPi-Klipper having both a name and a logo looking like OctoPi and the list getting randomly re-arranged. We have decided to change the name and logo because some users were getting confused. Related: guysoft/OctoKlipperPi#4 new json: {
"os_list": [
{
"name": "OctoKlipperPi",
"description": "Includes the OctoPrint host software for 3d printers and Klipper 3D printer firmware service",
"website": "https://github.com/guysoft/OctoKlipperPi",
"icon": "https://github.com/guysoft/OctoKlipperPi/raw/main/media/rpi-imager-OctoKlipperPi.png",
"subitems_url": "https://unofficialpi.org/rpi-imager/rpi-imager-octoklipperpi.json"
}
]
} |
The new snippet should be in now. |
Confirmed, closing :) |
Update: Fixed and tested
Hey,
I am the maintainer of OctoPi-Klipper and OctoPi.
OctoPi addition went pretty well, I wish to add OctoPi-Klipper too.
There is an issue with the manual install documented on Klippers issue tracker here: Klipper3d/klipper#4635
Publishing the images makes it accessiable as mainline OctoPi.
Following instructions as here: #162
Note that it should go in the
Other spesific purpose OS
section.And the json file is in place:
http://unofficialpi.org/rpi-imager/rpi-imager-octopi-klipper.json
The json file is built automatically using github actions now :)
Tested locally and it works
The text was updated successfully, but these errors were encountered: