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.

Sign up for GitHub

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

Jump to bottom

CPA calculation #398

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

CPA calculation #398

wants to merge 8 commits into from

Conversation

quantenschaum
Copy link
Contributor

see #397

@quantenschaum
reviewed and reworked ais CPA computation and evaluation
4193b53 
- use labels form ais formatter in AisTargetWidget
- display target name/MMSI in AisTargetWidget
- display TCPA in minutes
- highlight DCPA/TCPA/pass in AisInfoPage if over threshold
- correct for the age of the ais data before CPA computation
- do not set cpa=tcpa=0 if cpa is undefined (undefined!=0)
- set warning flag only if tcpa>0 (cpa not yet passed)
- set warning flag on all target over threshold
- reviewed cpa computation: looks good to me
- never set cpa=dst
- revised we-pass-logic
  - front (1) = we will cross track of target in front of target
  - back (-1) =  we will cross track of target astern of target
  - pass (0) = we have crossed the track of the target already
  - done (undefined) = we have crossed the track and have passed CPA
 - added BCPA = bearing to CPA
@quantenschaum
use downstream formatters for check for undefined values
57d6fca
@quantenschaum
AisInfoPage: adjusted field order
c9311fa
@quantenschaum
AisTargetWidget: show BRG to target instead of CPA if CPA has been pa…
9705a68 
…ssed
@quantenschaum
AisPage: adjusted displayed fields and alignment, added priority sort…
e2579ee 
…ing (combined DCPA/TCPA distance)
@quantenschaum
cleanup
07f9a67
@quantenschaum
relative motion vectors: correct RMVs for age of AIS data
3f7b350
@quantenschaum
improved sorting of ais targets
8aa7248 
- include target distance
- sort by distance for passed CPAs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@quantenschaum