Talk:Radio propagation: Difference between revisions

Revision as of 05:53, 13 March 2014

We are currently looking into updating the code base, to ensure that things still work with FG 3.0+ Some of the steps involved are detailed below:

integrate with navaid/radio code[1] Done
rebase against git HEAD (next) Done
rebase into a set of smaller self-contained patches [2] Not done
modify build system to make the feature entirely optional via build-time option that defaults to disabled [3] [4] Done
Address issues pointed out by TorstenD on the devel list: Not done
- generalize/unify with terrain presampler code [5] Template:Fixed: won't change
- get rid of huge conditionals, introduce helper functions/methods instead [6]
- get rid of magic constants, use ~~variables~~ enums [7] Done

provide a startup option to enable/disable the system [8] [9] [10] [11] Done
implement SGPropertyChangeListener to change the SGSubsystem and make the feature runtime-configurable Howto:Create new subsystems Not done
add a setting to the realism dialog[12] Not done
explore allowing aircraft developers to specify and customize a "radio equipment" profile for their aircraft (e.g. 747 vs. c172p antenna/power etc) [13] Not done
support different propagation models by coming up with an interface: Done
turn the whole thing into a dedicated SGSubsystem[14] Done
explore what is involved in allowing fgcom-standalone to use the same system [15] Done
investigate potential threading/HLA benefits Not done
investigate exposing a Nasal/CppBind interface for people working on features that benefit from having a radio propagation model Canvas Radar Not done
add unit tests to the cmake build system Not done

@@ Line 11: / Line 11: @@
 ** generalize/unify with terrain presampler code [https://www.mail-archive.com/flightgear-devel@lists.sourceforge.net/msg38927.html] {{Fixed: won't change}}
 ** get rid of huge conditionals, introduce helper functions/methods instead [https://www.mail-archive.com/flightgear-devel@lists.sourceforge.net/msg38927.html]
-** get rid of magic constants, use variables [https://www.mail-archive.com/flightgear-devel@lists.sourceforge.net/msg38927.html] {{Fixed: won't change}}
+** get rid of magic constants, use <strike>variables</strike> enums [https://www.mail-archive.com/flightgear-devel@lists.sourceforge.net/msg38927.html] {{Done}}
 * provide a startup option to enable/disable the system [https://www.mail-archive.com/search?l=flightgear-devel@lists.sourceforge.net&q=subject:%22Re%3A+%5BFlightgear-devel%5D+Real-Time+Radio+Propagation%22] [https://www.mail-archive.com/flightgear-devel@lists.sourceforge.net/msg38826.html] [https://www.mail-archive.com/flightgear-devel@lists.sourceforge.net/msg33722.html] [https://www.mail-archive.com/flightgear-devel@lists.sourceforge.net/msg38918.html] {{Done}}