Git Product home page Git Product logo

Comments (14)

oMTEo avatar oMTEo commented on August 21, 2024 1

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.

from knomi.

MasterTrandem avatar MasterTrandem commented on August 21, 2024

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.

from knomi.

MasterTrandem avatar MasterTrandem commented on August 21, 2024

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.

from knomi.

oMTEo avatar oMTEo commented on August 21, 2024

// "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

from knomi.

MasterTrandem avatar MasterTrandem commented on August 21, 2024

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

from knomi.

MasterTrandem avatar MasterTrandem commented on August 21, 2024

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.

from knomi.

MasterTrandem avatar MasterTrandem commented on August 21, 2024

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?

from knomi.

bigtreetech avatar bigtreetech commented on August 21, 2024

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

from knomi.

jgefaell avatar jgefaell commented on August 21, 2024

@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

from knomi.

Guilouz avatar Guilouz commented on August 21, 2024

@bigtreetech Always the same issue on KNOMI1

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

from knomi.

claudioguareschi avatar claudioguareschi commented on August 21, 2024

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

from knomi.

Coolbobjoe avatar Coolbobjoe commented on August 21, 2024

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!

from knomi.

Jinxted avatar Jinxted commented on August 21, 2024

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.

from knomi.

Jinxted avatar Jinxted commented on August 21, 2024

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.

from knomi.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.