Comment Search

minor typo : fix batteru

Add space in [registerSel]

break this line

after receiving the request for vitals send the response adn vitals data as well.

also please update verifying the request as this way will always pass

Update spelling of cummulativeRinsebackVolume_mL

Should these three alarms be moved after alarm 852?

It looks like 999 is currently placed before 852.

break this line

mvoe to utility

Fixed

BN approves

add a blank line above

Initialization is missing in the init function for variables from line 72 to 77.

Fixed

This was correct. Rever back the inputs and outputs.

Add to inputs: preTreatmentStatePublishInterval

break this line

Can we align like this
result = u32BroadcastIntervalOverride( message,
&preTreatmentStatePublishInterval,
TASK_GENERAL_INTERVAL );

break this line

assign this to a variable and then return the variable.

break this line

initialized using resetRinsebackFlags()

Removed the test scripts

can we define this as NUM_OF_BATTERY_REGISTERS = BATTERY_END_OF_LIST_TIME_TO_PUBLISH,

and use NUM_OF_BATTERY_REGISTERS everywhere instead of using [ NUM_OF_BATTERY_REGISTERS - 1 ]

Inputs : fpgaSensorReadings
Outputs: none

Using the lambda updated the function

When we send any message to UI, usually add the message id in the doxygen header like this.

@details \b Message \b Sent: MSG_ID_TD_RESP_TREATMENT_PARAMS_TO_VALIDATE

Please refer to the function sendTreatmentParamsResponse()