search

ludiazv / node-red-contrib-nrf24

NODE-Red fredom for nrf24l01(+)! Control nrf24l01(+) radios , RF24Mesh & RF24Gateway networks from Node-Red
MIT License
7 stars 4 forks source link

Default configuration does not work #2

Closed Zykino closed 4 years ago

Zykino commented 5 years ago

Default wiring provided by https://github.com/ludiazv/node-nrf24 and default conf of this module result in this line appearing every 2s. RF24 HARDWARE FAIL: Radio not responding, verify pin connections, wiring, etc.

node red web interface is not responding. I don't know how to fix this (start node red without it's flows)

Raspi 3B+ Raspbian Stretch Lite from April 2019 (updated/upgraded).

ludiazv commented 5 years ago

Hi, This error is reported directly buy the RF24 library. This is typcally due to wiring problems or lack of power in the module. You need to adapt the wiring to your configuration if this is not the case. I will need some detail to track the issue.

Zykino commented 5 years ago

Hi,
As said in #3, I followed the wiring instructions of the base package. But the instructions point toward setting the CE parameter to 22, not 25 like the current default of this package (1st problem).

2nd problem: Once a bad config is deployed, node-red interface is unusable, web interface and node-red-stop doesn't work. I don't know how to start node red web interface without starting the deployed flow. So a user need to go in the red node config file and fix the config by hand. I'm sure it's possible to get the error and stop trying to send the message. Or do something so node-red does not freeze.

ludiazv commented 5 years ago

Hi, 1st. the CE pin is selectable by the user according to your wiring you can use any GPIO. The documentation uses 22 and 25 just as examples. I typically use 22 & 24 for CE lines but this is because I use a custom board that uses this two pins but you can change it. This is why they are mandatory to create the radio configuration node or in the constructor in the base library.

2nd: I will do some tests to avoid this problem. I hope to have some answers soon.

ludiazv commented 5 years ago

Hi, I discover the problem for the 2nd point here. I have to test it but i think I can release a new version in the comming days to manage faliure situations. The problem is on how the base library is compiled.

ludiazv commented 5 years ago

Hi @Zykino , I included in the master branch a test during radio init to check if wiring is OK. Now it do not hang node-red and report an error. If interested check it out. This closes the 2nd issue. I added a new feature to implement auto recovery of hardware failure transparent to the user. This is experemintal but can be enabled in the config node checking the flag to activate auto recovery.

ludiazv commented 5 years ago

I need to update the docs and a final test. I will try during the weekend.

El 16 jun 2019, a las 9:27 a. m., Oliver Hruby notifications@github.com escribió:

Hi, will there be a new npm some time soon for the fixes? Version 0.0.2 is not showing any configuration node (just a textbox with no way to expand to configure it) so it's not possible to use it. I'm not sure if this is related to the issues mentioned in this thread.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or mute the thread.

oliverhruby commented 5 years ago

Sorry this time it was my problem :) I didn't check the node-red logs, I had issues with RF24 package, after fixing that the configuration node was showing ok.

Thanks for this library, you're doing a great job. One minor suggestion - think about putting a Donate icon on your github readme, people can send you something if they find your library useful.

Oliver

Dňa po, 17. jún 2019 10:24 AM ludiazv notifications@github.com napísal(a):

I need to update the docs and a final test. I will try during the weekend.

El 16 jun 2019, a las 9:27 a. m., Oliver Hruby notifications@github.com escribió:

Hi, will there be a new npm some time soon for the fixes? Version 0.0.2 is not showing any configuration node (just a textbox with no way to expand to configure it) so it's not possible to use it. I'm not sure if this is related to the issues mentioned in this thread.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ludiazv/node-red-contrib-nrf24/issues/2?email_source=notifications&email_token=ABOVQPBOI6ZLW4LX6GCZ53LP25C5RA5CNFSM4HKVVYA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX2NJVQ#issuecomment-502584534, or mute the thread https://github.com/notifications/unsubscribe-auth/ABOVQPFAI6TWHMMUMFHHYI3P25C5RANCNFSM4HKVVYAQ .

ludiazv commented 5 years ago

Great. I will do a test while I finish the documentation. Thanks for the feedback.

As regards "Donate button". Frankly I do not know how to do it. any suggestions?