Pressures.h

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
This is a msg payload structure. Even though all fields are 4 bytes, we should still pack it in case we add smaller fields later.

This is a msg payload structure. Even though all fields are 4 bytes, we should still pack it in case we add smaller fields later.

Removed

Removed

Updated

Updated

Not required I guess

Not required I guess

There is no override for the TMP long filtered pressure value. Please add a new one for it.

There is no override for the TMP long filtered pressure value. Please add a new one for it.

Please update the documentations too.

Please update the documentations too.

Remove blank line.

Remove blank line.

Now I don't see .data being set anywhere. Shouldn't we have tmpPressure.data = tmpPres;

Now I don't see .data being set anywhere. Shouldn't we have tmpPressure.data = tmpPres;

Updated

Updated

Passed tmpPres

Passed tmpPres

Updated

Updated

Removed and updated

Removed and updated

Need to calculate this before filtering function above so it can be passed into the function.

Need to calculate this before filtering function above so it can be passed into the function.

Need to pass in TMP to this function now.

Need to pass in TMP to this function now.

Shouldn't use .data here. TMP should be passed into this function like art and ven pressures and we should use the parameter here instead of .data.

Shouldn't use .data here. TMP should be passed into this function like art and ven pressures and we should use the parameter here instead of .data.

This looks wrong.

This looks wrong.

LEAHI-TD-FIRMWARE-LDT-1415_Pressures Monitoring - TD
LEAHI-TD-FIRMWARE-LDT-1415_Pressures Monitoring - TD
Commented out already but I dont know why it is showing the same

Commented out already but I dont know why it is showing the same

It's still UI only. Shouldn't this be commented out before we merge?

It's still UI only. Shouldn't this be commented out before we merge?

Yes, updated

Yes, updated

Removed

Removed

Remove blank line.

Remove blank line.

Looks like this reverted back.

Looks like this reverted back.

LEAHI-TD-FIRMWARE-LDT-1415_Pressures Monitoring - TD
LEAHI-TD-FIRMWARE-LDT-1415_Pressures Monitoring - TD
Yes, we do and this is not required here so, I removed

Yes, we do and this is not required here so, I removed

Shouldn't we be getting these defaults from the TxParams unit?

Shouldn't we be getting these defaults from the TxParams unit?