application

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-6359: Addressed code review comments.

Merge branch 'DEN-5777_UI_DEV_ConvertLogging_From_hrs_to_%' into UI_DEV

DEN-6619: Implementation

- Added a signal named didSDCardSpaceTooLow in the DriveWatcher which will be triggered when the free storage space available on the SD-CARD is less than 10%. (No action on UI happens currently)

- Added a flag in DriveWatcher to stop emitting the Logger while it is cleaning the Logs [[ IMPORTANT : is is handled by signal/slots for thread synchronization and safety ]].

- Added a function to find the total size of the file in a path.

- Added find function to find files with a list of filter in a path.

- Added an overloaded version of find to help find the files in a path which are exceeding the storage limit from the oldest files.

- Added error check in the read overloaded which works with Json.

- Re-implemented the FileHandler::removeLogs to clean up the logs by % instead of date.

- Removed unused global log paths from StorageGlobal.

-

    • -5
    • +130
    /sources/storage/FileHandler.cpp
Merge branch 'DEN-5751_Pre-TreatmentUltrafiltration' into UI_DEV

Can delete

Can delete

Do these properties still exist?

Do these properties still exist?

If FW responds back saying either arterial or venous pressure limit ranges are invalid, the user corrects them, and then we proceed, without doing anything upon validation success the pressure limi...

If FW responds back saying either arterial or venous pressure limit ranges are invalid, the user corrects them, and then we proceed, without doing anything upon validation success the pressure limits would still show up as invalid the next time the user enters the create treatment screen.

How will this notification bar look if an alarm is raised but minimized? Would there be two notification bars visible?

How will this notification bar look if an alarm is raised but minimized? Would there be two notification bars visible?

It seems like some variables are being defined in QML while others are in cpp. I've seen this happen with TreatmentCreate. What's the plan for where PRS default variables should be defined?

It seems like some variables are being defined in QML while others are in cpp.
I've seen this happen with TreatmentCreate. What's the plan for where PRS default variables should be defined?

Can delete

Can delete

Should point to HDSimulator since denaliMessages has been deprecated

Should point to HDSimulator since denaliMessages has been deprecated

Should point to HDSimulator, as denaliMessages has been deprecated

Should point to HDSimulator, as denaliMessages has been deprecated

Should point to the HDSimulator

Should point to the HDSimulator

This should point to the HDSimulator to avoid future conflicts with dialin

This should point to the HDSimulator to avoid future conflicts with dialin

Should point to the HDSimulator

Should point to the HDSimulator

here as well (see above)

here as well (see above)

These limits should reside next to the other treatment parameter defaults in TreatmentCreate.h. Putting them in QML causes changes to the treatment parameters configuration file (specifically arter...

These limits should reside next to the other treatment parameter defaults in TreatmentCreate.h.
Putting them in QML causes changes to the treatment parameters configuration file (specifically arterial and venous pressure limits) to have no effect.

This has been removed from dialin

This has been removed from dialin

denaliMessages has been deprecated. Please don't add it back to dialin. This should be self.hd_simulator.alarms.cmd_set_alarm_triggered

denaliMessages has been deprecated. Please don't add it back to dialin. This should be

self.hd_simulator.alarms.cmd_set_alarm_triggered
denaliMessages has been deprecated. Please don't add it back to dialin. This should be self.hd_simulator.alarms.cmd_set_alarm_cleared

denaliMessages has been deprecated. Please don't add it back to dialin. This should be

self.hd_simulator.alarms.cmd_set_alarm_cleared
Should be self.hd_simulator.alarms.cmd_activate_alarm_id

Should be

self.hd_simulator.alarms.cmd_activate_alarm_id
Should be self.hd_simulator.set_treatment_heparin_data(volume)

Should be

self.hd_simulator.set_treatment_heparin_data(volume)
DEN-5758: Code Analysis

Merge branch 'master' into DEN-5736_UI_Heparin_Bolus

will be done later. RESOLVED

will be done later.
RESOLVED

needs to investigate more or done later. RESOLVED

needs to investigate more or done later.
RESOLVED

will be done later. RESOLVED

will be done later.
RESOLVED

will be done later. RESOLVED

will be done later.
RESOLVED

Resolved as per our code review 1.27.2021

Resolved as per our code review 1.27.2021