Skip to content

Conversation

kasbert
Copy link
Contributor

@kasbert kasbert commented Oct 22, 2021

Decoder for Telldus weather station indoor unit.

As the indoor unit receives a message from the outdoor unit, it sends 3 radio messages

  • Oregon-WGR800
  • Oregon-THGR810 or Oregon-PCR800
  • Telldus-FT0385R (this one)

Barometric pressure and indoor huminity are only in this message

@zuckschwerdt
Copy link
Collaborator

So this "Telldus-FT0385R" is basically like the "Cotech 36-7959" ("SwitchDocLabs FT020T") with 36 bytes instead of 13, right? Can you rewrite this PR based on cotech_36_7959.c instead please?

@zuckschwerdt
Copy link
Collaborator

Better to output "temperature_F" and use the -C si option.

}
return crc;
}

Copy link
Owner

Choose a reason for hiding this comment

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

The crc stuff should be using the native functions. If you post ca 10 message we should be able to figure out what to use instead of this code.

Copy link
Owner

Choose a reason for hiding this comment

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

Maybe it uses the same as the cotech code.

@zuckschwerdt
Copy link
Collaborator

I verified it to be the same. This PR should base on cotech_36_7959.c

@kasbert
Copy link
Contributor Author

kasbert commented Oct 25, 2021

There is now a new version

@zuckschwerdt
Copy link
Collaborator

Looks good. (the DECLneeds to be the last in the list)

So the indoor temp has a range of ~16.7 C to ~31 C only, with precision of ~0.05 C?

@kasbert
Copy link
Contributor Author

kasbert commented Oct 26, 2021

I fixed the indoor temperature.

Manual says that Indoor Temperature Measuring Range: .............................................................0 – +60°C

Wasn't DECL(telldus_ft0385r) in the right place ?

@zuckschwerdt
Copy link
Collaborator

Wasn't DECL(telldus_ft0385r) in the right place ?

Sorry, maybe I saw that wrong. Always adding at the bottom ist correct.

@zuckschwerdt zuckschwerdt merged commit 098872c into merbanan:master Dec 10, 2021
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.

3 participants