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

Added Tx20 Wind Sensor #3146

Closed
wants to merge 3 commits into from
Closed

Conversation

TheEggi
Copy link

@TheEggi TheEggi commented Jul 6, 2018

Details of the sensor can be found in this issue:
#2654

The wind sensor consist of:

  • red cable (vcc)
  • yellow (gnd)
  • black cable (transmit) - can be configured using UI
  • green cable (starts transmission - should be connected to gnd)

Config:
image

Result:
image

Json Result:
"TX20":{"SPEED_AVG:":nan, "SPEED_MAX":0.00, "DIRECTION":"NNE"}

@Jason2866
Copy link
Collaborator

Hi @TheEggi sounds interesting 👍
How precise is the TX20? (Schätzeisen oder Messgerät).
TXH and greets from South Germany

@TheEggi
Copy link
Author

TheEggi commented Jul 9, 2018

To be honest ... I don't have anything that I could compare it to. I mean the position is marked on the sensor and seems to work as expected but for the wind speed the only test that I could do was to blow at it and see if the speed is increasing/decreasing - and that seemed to have worked. If you have a better way/idea (without any real testing equipment) how I can test the precision of the device, I am more than happy to try it.

@Jason2866
Copy link
Collaborator

Jason2866 commented Jul 9, 2018

I think i buy one. I can compare it with the quality (expensive) Aneometer from my brother (hang gliding...)
No really idea for testing, maybe drive with a bike with portable mqtt equpment...
Thx

@Silencer80
Copy link

Thank you very much @TheEggi for implementing this!

I have just returned from a businesstrip and have a lot of work to do. However, I already have a NodeMCU and a TX20 sensor on my desk to test that as soon as possible.

I'm so curious if that works well. I can compare the results with my mdt SCN-WS3HW.01 Weatherstation.

Thank you again!
-Boris

@FabianHM
Copy link

FabianHM commented Oct 4, 2018

Hello,
What are the chances that Tasmota's support will be short-term? I could buy the device just under 30 € but would not buy it without Tasmota use.
Greetings Fabian

@Jason2866
Copy link
Collaborator

@FabianHM
Dont hesitate. Buy one it works with the fork from @TheEggi

@@ -2551,4 +2551,5 @@ void loop()

// yield(); // yield == delay(0), delay contains yield, auto yield in loop
delay(sleep); // https://github.com/esp8266/Arduino/issues/2021
XsnsCall(FUNC_LOOP);
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi, This line shouldn't be there.

Please, check other sensors to see the way tasmota handle this.

/*********************************************************************************************\
* Tx20 wind sensor
\*********************************************************************************************/

Copy link
Contributor

Choose a reason for hiding this comment

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

You need to add a

Suggested change
#ifdef USE_TX20_wind_sensor

in order to enable or disable this sensor from user_config.h file

By default this sensor should be disabled in user_config.h

#endif // USE_WEBSERVER
}
return result;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Add below this a
#endif // enable sensor

@ascillato
Copy link
Contributor

@TheEggi

This PR has been rebased with the new version of Tasmota.

Could you update it?

@FabianHM
Copy link

Details of the sensor can be found in this issue:
#2654

The wind sensor consist of:

* red cable (vcc)

* yellow (gnd)

* black cable (transmit) - can be configured using UI

* green cable (starts transmission - should be connected to gnd)

Config:
image

Result:
image

Json Result:
"TX20":{"SPEED_AVG:":nan, "SPEED_MAX":0.00, "DIRECTION":"NNE"}

What does the analogue input in screenshot "10" indicate?

@ascillato
Copy link
Contributor

Means that in his version he has enabled the use of the Analog input on the pin A0 but nothing is connected there. Just that. Nothing to do with this PR nor the sensor. Don't worry.

@TheEggi
Copy link
Author

TheEggi commented Oct 20, 2018

Added most of your suggestions. Maybe you have a suggestion on which sensor I could check as an example to remove the FUNC_LOOP.

@arendst
Copy link
Owner

arendst commented Oct 20, 2018

Based on your work I will rewrite and release soon. You then have the opportunity to test as I do not own a Tx20...

@arendst
Copy link
Owner

arendst commented Oct 20, 2018

I see that you translate D_UNIT_KMH into "kpH". Shouldn't that be "kmh"?

@TheEggi
Copy link
Author

TheEggi commented Oct 20, 2018

I guess kmh is the correct short form of kilometer per hour – so you are right.

@Jason2866
Copy link
Collaborator

Hello together,
i have bought a TX20 this week. Happy to test when the "final" version is done.

arendst added a commit that referenced this pull request Oct 21, 2018
Add support for La Crosse TX20 Anemometer (#2654, #3146)
@arendst arendst closed this Oct 21, 2018
@arendst
Copy link
Owner

arendst commented Oct 21, 2018

Just released a version for the TX20. Give it a try and let me know if it works pls.

@TheEggi
Copy link
Author

TheEggi commented Oct 21, 2018

Seems to be working. Thank you for adding it.

@Jason2866
Copy link
Collaborator

Thx for programming and adding.
image

@arendst
Copy link
Owner

arendst commented Oct 28, 2018

Thx for testing ;-)

@Sepultura22
Copy link

Hi, I have a doubt. I bought a TX20 sensor and it does not work for me. I have loaded the latest tastmota firmware in a basic sonoff and soldered the black cable in gpio4, the yellow and green cable in gnd and the red in vcc. I have also selected in the configuration that gpio4 acts with TX20. Should we do more? Thank you very much for your work, it is just what I was looking for.

@travelling-man
Copy link

travelling-man commented Dec 8, 2018

Hello,

I have a problem that only updates the wind direction. The speed doesn't change and stays at 0.0 km/h permanently.

Does anyone have similar problems?

bildschirmfoto 2018-12-08 um 11 39 36

@Jason2866
Copy link
Collaborator

Nope, works fine for me

@FabianHM
Copy link

FabianHM commented Dec 8, 2018

it works fine for me too. No Problems

@travelling-man
Copy link

travelling-man commented Dec 8, 2018

what hardware do you use?

All cables (red -> 3,3v) (green, yellow -> gnd) (black -> d2) are already connected before starting the Wemos.

I am using a Wemos d1 mini and its 3,3v.

Strange is that the wind direction is updated.

Is there a way to debug it?

@Jason2866
Copy link
Collaborator

20181208_144849
Just this...

@Jason2866
Copy link
Collaborator

Jason2866 commented Dec 8, 2018

Isnt installed because solar charging part isnt done right now...
3d printend stuff needs to be done too

@travelling-man
Copy link

How long is the cable from your TX-20? I had some measurement results for a few seconds yesterday.

@TheEggi
Copy link
Author

TheEggi commented Dec 10, 2018

I cut the cable down to ~1.5m as I did not need it to be any longer - worked without any issues.

@travelling-man
Copy link

I cut the cable down to ~1.5m as I did not need it to be any longer - worked without any issues.

Can you take a picture with the model number?

@travelling-man
Copy link

Does anyone still have a photo with the original pin assignment?

@davide5681
Copy link

e possibile direttamente da tasmota abilitare un uscita una volta arrivati ad una soglia di velocità ?

@Pastant
Copy link

Pastant commented Feb 8, 2020

I tried the TX23 because the TX20 is not available any longer. The TX23 is not compatible! Even the support for the TX20 was dropped in recent releases (8.1) ...

@Jason2866
Copy link
Collaborator

Support for TX20 is NOT dropped. It is just not in a precompiled version (as many other drivers)
Read section self compile in the documentation www.tasmota.com

@Pastant
Copy link

Pastant commented Feb 8, 2020

The "Complete list of available feature and sensors." states that tasmota-sensors.bin supports the TX20. So the precompiled supprt is dropped. I won't compile a version because I only own a TX23 which is not compatible ...

@curzon01
Copy link
Contributor

I'm currently trying to adapt the TX20 driver also for TX23. The protocol is the same but TX23 needs an initial signal to send out their data where TX20 sends every 2 seconds automatically.

curzon01 added a commit to curzon01/Tasmota that referenced this pull request Feb 20, 2020
arendst added a commit that referenced this pull request Feb 20, 2020
Add support for La Crosse TX23 Anemometer (#3146)
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.