Include flight message age in aircraft lead time #44
Conversation
This file contains 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
Validation Issues Found (Python) 🐍File: edge-detect/ai/datasets2.py Rule: |
| a_h = currentPlane["altitude"] # [m] | ||
| # currentPlane["altitudeTime"] | ||
| # currentPlane["altitudeTime"] # Expect altitudeTime to equal latLonTime |
There was a problem hiding this comment.
@ralatsdc - the altitudeTime is not equal to the latLonTime. They come in as separate ADS-B messages and can add up to some sizeable differences if messages get missed. Can you split these out appropriately?
|
The altitudeTime and the latLonTime In the two test tracks are identical. Is this an exception? If it is, then it would be helpful to know at which time the groundSpeed and verticalRate correspond. Then linear interpolation could be used to adjust longitude, latitude, and altitude to a single time.
…On Feb 9, 2023, 4:24 PM -0500, mchadwick-iqt ***@***.***>, wrote:
@mchadwick-iqt commented on this pull request.
In axis-ptz/camera.py:
> a_h = currentPlane["altitude"] # [m]
- # currentPlane["altitudeTime"]
+ # currentPlane["altitudeTime"] # Expect altitudeTime to equal latLonTime
@ralatsdc - the altitudeTime is not equal to the latLonTime. They come in as separate ADS-B messages and can add up to some sizeable error if messages get missed. Can you split these out appropriately?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
|
@ralatsdc there are a couple of ADS-B feeds we are working with. The test data came from one system (PingUSB). There is a separate system (which is more accurate) which has different times for altitude and lat/long. |
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.
Note that for testing an environment variable INCLUDE_AGE must be set to "False". By default, age of the flight message is included in computing the aircraft lead time.