KNOMI icon indicating copy to clipboard operation
KNOMI copied to clipboard

No saving and connection

Open MasterTrandem opened this issue 1 year ago • 14 comments

Hello,

I have the problem that the Knomi 2 don't save the wifinetwork. After reboot comes the Hello logo back. I change the ssid length but thats not helping. He only save the Ip from klipper.

The other problem is that Knomi says the printer is not opartional. When i reboot klipper you can see that Knomi have connection and show the reboot animation, but after thsn nothing. I have a Sonic Pad.

MasterTrandem avatar Dec 01 '23 04:12 MasterTrandem

Okay i found the problems. 1. The length of the moonrake_ip is the problem. Sonic Pad give every printer a own port. That means for section of the ip adress needs more len than 16. The other problem is the restore of the EEPROM. When it restore the password comes like this �zBPZ�K��� out of the EEPROM.

MasterTrandem avatar Dec 01 '23 20:12 MasterTrandem

So i thing i fix the problems. For the moonraker IP i change in the config.h String moonraker_ip; // "192.168.255.255" max len = from 16 to 22. For the reboot Problem i change 2 things. First i write in the wifi_setup.cpp direct my SSID and password at .sta_ssid und .sta_pwd. Some was writing when he change the lengt iof the SSID the saves in the EEPROM work, because of this point i change in the wifi_setup.cpp if (!EEPROM.begin()) from 1024 to 4096. I thing the space was not enouth.

MasterTrandem avatar Dec 02 '23 11:12 MasterTrandem

// "192.168.255.255" max len = 16 its just a comment and wont change anything

(!EEPROM.begin()) from 1024 to 4096 Ive tried this, but also doesnt fix this bug

oMTEo avatar Dec 02 '23 19:12 oMTEo

Okay than i don't know. Sorry yes 😂 i forgot to say that i write the ip also inside the wifi_setup, but on the website i can see more than 16. I make this with ChatGPT and he say to change this 16 to 22. Screenshot 2023-12-02 221155

MasterTrandem avatar Dec 02 '23 21:12 MasterTrandem

ChatGPT.... XD, next time ask someone with basic programming skills :D

This :7125 is Port-Direction and dont need to be written, only IP.

An IP -> xxx.xxx.xxx.xxx is 15 characters, so a length of 16 is enough.

oMTEo avatar Dec 03 '23 04:12 oMTEo

Yes but this was easier. No i need the port for the Sonic Pad, because you can use 4 printer simultaneously. Every printer gets his own port. When i only give Knomi the IP he says that the printer is unoperative.

MasterTrandem avatar Dec 03 '23 07:12 MasterTrandem

Do

// "192.168.255.255" max len = 16 its just a comment and wont change anything

(!EEPROM.begin()) from 1024 to 4096 Ive tried this, but also doesnt fix this bug

Do you make a hard reset?

MasterTrandem avatar Dec 03 '23 19:12 MasterTrandem

Hi Please try the latest version of firmware, this should solve the bug of not saving WiFi when password or name are long. KNOMI2: https://github.com/bigtreetech/KNOMI/blob/master/KNOMI2/Firmware/knomi2_firmware.bin KNOMI1: https://github.com/bigtreetech/KNOMI/blob/master/KNOMI1/Firmware/knomi1_firmware.bin Follow the steps below to update: https://github.com/bigtreetech/KNOMI/#firmware-update

bigtreetech avatar Dec 06 '23 02:12 bigtreetech

@bigtreetech Thank you for addressing the Wifi saving defect. Can you post a version of the firmware that fixes the length limitation for the IP address allowing us to set discrete TCP ports for the printer?

Opened #48

jgefaell avatar Dec 07 '23 05:12 jgefaell

@bigtreetech Always the same issue on KNOMI1

My SSID have 14 characters and my password have 23 characters.

Guilouz avatar Dec 08 '23 22:12 Guilouz

Hi Please try the latest version of firmware, this should solve the bug of not saving WiFi when password or name are long. KNOMI2: https://github.com/bigtreetech/KNOMI/blob/master/KNOMI2/Firmware/knomi2_firmware.bin KNOMI1: https://github.com/bigtreetech/KNOMI/blob/master/KNOMI1/Firmware/knomi1_firmware.bin Follow the steps below to update: https://github.com/bigtreetech/KNOMI/#firmware-update

@bigtreetech The new firmware still has a problem of not saving the web interface data correctly. The issue is in wifi_setup.cpp in the eeprom write function:

EEPROM.put<knomi_config_t>(0x00 + sizeof(EEPROM_SIGN), knomi_config);

The size of EEPROM_SIGN (4 bytes) is not returned correctly by sizeof(EEPROM_SIGN).

Change to EEPROM.put<knomi_config_t>(0x00 + EEPROM_SIGN_SIZE, knomi_config); as it was correctly done in the rest of the code.

The older firmware version worked just fine after this change, without the need to convert all the strings to char[].

The code of the previous firmware version looked a lot cleaner and more maintainable

claudioguareschi avatar Dec 09 '23 01:12 claudioguareschi

Has this been fixed? I tried the stock firmware, no dice, i tried flashing the new firmware nothing, so far ive had 0/4 brand new BTT products have limited or absolutely no functionality. Getting close to the point of sending back the entire shipment!

Coolbobjoe avatar Dec 17 '23 16:12 Coolbobjoe

This has not been fixed in the most current firmware. The issue is we need to be able to enter the port. Not everyone is using a printer that comes stock with vanilla Klipper and Moonraker. People using printers like the K1 Series or the Sonic Pad are stuck needing to have a port set in order to communicate with Moonraker appropriately. Without the ability to tell the Knomi which port to access, we have paperweights. I wish I had not updated my Knomi firmware because the ancient version allowed the port to be set within the ip address. Now I have a Knomi I cannot use.

Jinxted avatar Dec 26 '23 16:12 Jinxted

I take that back - for the v1, the link in the readme and such that points to the "latest" firmware has not been updated to include the actual newest firmware release. I flashed the firmware from https://github.com/bigtreetech/KNOMI/tree/master/KNOMI1/Firmware and it now has a field for the port and is functioning correctly. Update to the newest firmware in the actual repository and do not use any of the current links that download a pre-zipped file with existing older firmware.

Jinxted avatar Dec 26 '23 17:12 Jinxted