add longitude/ latitude to support ttnmapper.org#1
Open
theArcher73 wants to merge 3 commits intoSeeed-Solution:mainfrom
Open
add longitude/ latitude to support ttnmapper.org#1theArcher73 wants to merge 3 commits intoSeeed-Solution:mainfrom
theArcher73 wants to merge 3 commits intoSeeed-Solution:mainfrom
Conversation
Longitude/ latidtude added to decoded_payload in order to use ttnmapper.org
Update SenseCAP_T1000_TTN_Decoder.js
Contributor
|
Hi, thanks for creating a PR to contribute to this decoder, but it looks like you added the wrong place, should add it on line 36: |
Author
|
Hi,I have only copied the two lines from my TTN into the code of the fork of your decoder to send them to you. Either I was lucky and the two lines work there too (which I don't really think) or I slipped in one line while pasting.Anyway, you know your code better, put it where it fits and have fun!Afterwards I thought, you could also deliver the code commented, so with one line above:// to use the TTNMapper.org please remove the comment for the following lines codeSteffenVon meinem iPhone gesesetAm 13.10.2023 um 04:34 schrieb Jessie ***@***.***>:
Hi, thanks for creating a PR to contribute to this decoder, but it looks like you added the wrong place, should add it on line 36:
for (let element of message) {
if (element.errorCode) {
decoded.err = element.errorCode
decoded.errMessage = element.error
} else {
if (element.measurementId === '4197') {
decoded.longitude = element.measurementValue
}
if (element.measurementId === '4198') {
decoded.latitude = element.measurementValue
}
elements.push(element)
}
}
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
|
|
Hi, the Problem with ttnmapper is that we need a value for precise of the data with satellite count or accuracy value. So if you have lat and lon its even not working. Sensecap needs a firmware update and send this data too. My Wish then: Height, Satellite Count and HDOP etc. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.