"Final Target Ultrafiltration Rate" for simplicity, the "Final" portion has been removed and only the "Target" is used since all SW knows them as "Target" only.
After talking with Hung, This change is currently on his feature branch and should not be changed in here, later which codes are merged it will be resolved in case of conflict.
Please rename this observer class to something more specific. Ex: NVUtilsObserver. Generalized naming may cause problems with advanced usage of the API.
I would set this to at least 11 because you are using sprintf to create an unsigned integer string (max is 10 digits + 1 zero terminator) and we want to make sure we give sprintf a large enough buffer to populate - otherwise it will overwrite memory.
That is a very good question. As much as possible we have to put the #includes in the CPP files and not the header. The #include in the header will always be imported even if you don't compile the code (only CPP is compiled in c, not the header) as opposed to just including when needed to compile in CPP. This also helps the compiler to optimize the code better and faster to compile. you only need to put it in the header if you inherit from the class defined in the class included or you have a variable/parameter defined of that file in the header.
This a matter of trying the temperature sensors more than once before we start the cycle. Making the wait time longer will not necessarily help the sensors to read within the range.
