I am (finally) submitting my pull request for the Audible Alerts 2.0 feature that I demo'ed a few days ago, incorporating just a few feedback items I got from that demo. I am linking to a walkthrough video I did of the code changes below, as I think that is often the easiest way to demo the tech changes (short of an interactive walkthrough, which I am happy to do as well). In summary, the following are the changes:
Many more "callout" options: vertical speed, distance (with optional decimal precision), various traffic Id options (including none or full callsign), both colloquial and individual digit number options
Configurable speaking speed (which does affect pitch); this is one that I added since the demo
Ground traffic and low-speed filter options
Ground traffic filter option for the location (visual) view as well; should have included this in the demo, as it was there, but forgot to
Many, many optimizations to the CPU usage, mainly by employing smarter (non-busy) waiting and avoiding premature notification of waiting threads
Factoring new features into small (private or final) methods, for JIT and AOT compilers to either inline or convert to native code, based on profile
Preference screen cleanup
Here is the walkthrough video of how these were implemented, focusing on the core codebase changes and any items that changed since the demo:
I am (finally) submitting my pull request for the Audible Alerts 2.0 feature that I demo'ed a few days ago, incorporating just a few feedback items I got from that demo. I am linking to a walkthrough video I did of the code changes below, as I think that is often the easiest way to demo the tech changes (short of an interactive walkthrough, which I am happy to do as well). In summary, the following are the changes:
Here is the walkthrough video of how these were implemented, focusing on the core codebase changes and any items that changed since the demo:
https://drive.google.com/file/d/1FV8jNPj8-sky3ZKehIaHVuIDoSvvlXAU/view?usp=sharing
Addresses #371 (both visual and audible traffic alerts allow filtering of ground traffic)