DIALIN-DEN-16454

DIALIN-DEN-16454

Activity on this page:

Line History

Line history chart not available

Line history chart not available

 

Stats

Commits this week: 0
Total Commits: 3,304
Last Commit: 16 Dec
Commits this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 
Friday 20 Oct 2023
10:12 am

Behrouz NematiPour

10:11 am

Behrouz NematiPour and msuleiman  (deleted user)

Thursday 19 Oct 2023
4:16 pm

Michael Garthwaite

4:16 pm

Michael Garthwaite

Resolved.

Resolved.

4:16 pm

Michael Garthwaite

Resolved.

Resolved.

4:15 pm

Michael Garthwaite

Resolved.

Resolved.

4:14 pm

vduong  (deleted user)

Done.

Done.

4:14 pm

vduong  (deleted user)

All set. I have reverted it

All set. I have reverted it

4:08 pm

Behrouz NematiPour

good point. Vy will fix it.

good point.
Vy will fix it.

4:07 pm

Behrouz NematiPour

Michael Garthwaite Ui code is frozen and we cannot change the UI code, so matched with what currently exists in UI. The attempt was to fix the integration tests, and we did not go over all the mess...

Michael Garthwaite
Ui code is frozen and we cannot change the UI code, so matched with what currently exists in UI.
The attempt was to fix the integration tests, and we did not go over all the messages API to fix them.
We can have it done later when have more time.

3:42 pm

Michael Garthwaite

If the UI can ignore the fields, then why does dialin need to match the UI?

If the UI can ignore the fields, then why does dialin need to match the UI?

3:41 pm

Behrouz NematiPour

UI will ignore the last two values, which is not using it. FW can add extra fields, but for now if UI don't use it or does not have to use it will just ignore.

UI will ignore the last two values, which is not using it.
FW can add extra fields, but for now if UI don't use it or does not have to use it will just ignore.

3:38 pm

Michael Garthwaite

this message structure is in Release FW. What is the UI going to with the incorrect message size?

this message structure is in Release FW. What is the UI going to with the incorrect message size?

3:37 pm

Behrouz NematiPour

Michael Garthwaite we needed to match it with UI and since UI code is freeze.

Michael Garthwaite
we needed to match it with UI and since UI code is freeze.

3:35 pm

Michael Garthwaite

Missing two fields. Current DG FW has MSG_ID_DG_HEATERS_DATA = 0x2C as /// Heaters data structure. typedef struct F32 mainPrimayHeaterDC; ///< Main primary heater DC F32 smallPrimaryHeaterDC; ///...

Missing two fields. Current DG FW has MSG_ID_DG_HEATERS_DATA = 0x2C as

/// Heaters data structure.
typedef struct

F32 mainPrimayHeaterDC; ///< Main primary heater DC
F32 smallPrimaryHeaterDC; ///< Small primary heater DC
F32 trimmerHeaterDC; ///< Trimmer heater DC
F32 primaryTargetTemp; ///< Primary heater target temperature
F32 trimmerTargetTemp; ///< Trimmer heater target temperature
U32 primaryHeaterState; ///< Primary heater state
U32 trimmerHeaterState; ///< Trimmer heater state
F32 primaryEfficiency; ///< Primary heater efficiency
F32 primaryCalcTargetTemp; ///< Primary heater calculated target temperature
F32 trimmerCalcCurrentTemp; ///< Trimmer heater calculated current temperature
U32 trimmerUseLastDC; ///< Trimmer heater use last duty cycle
F32 previsouFlow; ///< Trimmer heater previous flow L/min
U32 controlCounter; ///< Trimmer heater control count

HEATERS_DATA_T;

3:33 pm

msuleiman  (deleted user)

3:31 pm

Michael Garthwaite

These should not be commented out. Current FW builds have MSG_ID_PRESSURE_OCCLUSION_DATA = 0x9 with the following structure. /// Payload record structure for the pressure & occlusions data message...

These should not be commented out. Current FW builds have MSG_ID_PRESSURE_OCCLUSION_DATA = 0x9 with the following structure.

/// Payload record structure for the pressure & occlusions data message.

F32 arterialPressure; ///< Latest arterial pressure (mmHg)
F32 venousPressure; ///< Latest venous pressure (mmHg)
U32 bldPumpOcclusion; ///< Latest BP occlusion (no units)
U32 presLimitState; ///< Current pressure limits state (enum-stabilization or stable)
S32 artMinLimit; ///< Current arterial minimum pressure limit (mmHg)
S32 artMaxLimit; ///< Current arterial maximum pressure limit (mmHg)
S32 venMinLimit; ///< Current venous minimum pressure limit (mmHg)
S32 venMaxLimit; ///< Current venous maximum pressure limit (mmHg)
F32 arterialLongFilterPres; ///< Latest long filtered arterial pressure (mmHg)
F32 venousLongFilterPres; ///< Latest long filtered venous pressure (mmHg)
PRESSURE_OCCLUSION_DATA_T;

3:28 pm

Michael Garthwaite

Please remove blank line.

Please remove blank line.

Tuesday 26 Sep 2023
10:06 am

Dara Navaei

DIALIN-DEN-16454-UI-VD-S106 [ Final release integration tests fix ]
DIALIN-DEN-16454-UI-VD-S106 [ Final release integration tests fix ]