Add blank line between library includes and app includes. Library includes should come first, so FpgaDD.h above should come down into the app includes group and should have "" instead of <>. App includes should be in alphabetical order.
i believe they are.. this driver will have to be reviewed once we get the conductivity sensors onto the units.
getFPGAP9Data() pulls register 336 in the HDD that is 4 bytes. getFPGAP9Conductivity pulls register 340 that is 2 bytes. getFPGAP9Temperature pulls register 342 that is 2 bytes.
unsure what are the differences between getFPGAP9Data and the other two. Seem like they are the same
Not sure this should be overridable. Can still have Dialin set this, but with a set command (instead of an override command). If you allow override, can get into situation where target needs to be programmatically changed (e.g. zeroed when changing control method) and it only changes the .data and not the .ovdata and, if currently overridden, it's not zero and doesn't work right. General rule: -use override cmds for sensor readings and broadcast intervals (i.e. things that you get or are usually constant) -use set cmds for set points and targets (i.e. things that you set)
