application / sources

view

Clone Tools

Stats

Commits this week: 2
Total Committers: 12
Last Commit: 09 Jun
Commits this week: 2

Lines of code count not available

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
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 23 Sep 2021
4:32 pm

Behrouz NematiPour

DEN-10399: UI: Alarms block Shutdown screen

DEN-9271: Dialysate Temperature implemented resolution does not meet PRS

Options
    • -0
    • +1
    ./hd/alarm/VAlarmStatus.cpp
    • -0
    • +1
    ./hd/alarm/VAlarmStatus.h
  3. … 5 more files in changeset.
Wednesday 22 Sep 2021
2:53 pm

Behrouz NematiPour

Thanks for the comment. I removed those two and updated the comments as well.

Thanks for the comment.
I removed those two and updated the comments as well.

2:52 pm

Behrouz NematiPour

Updated the comments to clarify. In ' ..., and them,...' it meant logical AND &&. And added a lot more comments.

Updated the comments to clarify.
In ' ..., and them,...' it meant logical AND &&.
And added a lot more comments.

2:50 pm

Behrouz NematiPour

Removed.

Removed.

10:07 am

Sean Nash

Remove blank line.

Remove blank line.

9:52 am

Sean Nash

In the header file, the Ethernet and sound tests are said to be not required (i.e. should not prevent system use). So this comment appears to be inconsistent with the header file.

In the header file, the Ethernet and sound tests are said to be not required (i.e. should not prevent system use).
So this comment appears to be inconsistent with the header file.

9:50 am

Sean Nash

"... and then, and them," - looks like something is wrong with this comment.

"... and then, and them," - looks like something is wrong with this comment.

Tuesday 21 Sep 2021
6:26 pm

Sean Nash

RESOLVED.

RESOLVED.

6:26 pm

Sean Nash

RESOLVED.

RESOLVED.

5:55 pm

Behrouz NematiPour

In UI we have multiple types of TODOs. DEBUG, FIXME, WARNING, NOTE, TODO

In UI we have multiple types of TODOs.
DEBUG, FIXME, WARNING, NOTE, TODO

5:53 pm

Behrouz NematiPour

This was not a bug, It was a change request from Sarina to remove the Rinseback Aditional 5 min counter. I kept it in case we may need it later.

This was not a bug,
It was a change request from Sarina to remove the Rinseback Aditional 5 min counter.
I kept it in case we may need it later.

5:49 pm

Sean Nash

Does this mean commenting out these 2 lines fixed a bug? If so, why not just delete them?

Does this mean commenting out these 2 lines fixed a bug? If so, why not just delete them?

5:48 pm

Sean Nash

Temporary? Maybe add a // TODO or is // DEBUG something you'll know to search for later?

Temporary? Maybe add a // TODO or is // DEBUG something you'll know to search for later?

11:38 am

Behrouz NematiPour

updated.

updated.

11:38 am

Behrouz NematiPour

updated.

updated.

11:38 am

Behrouz NematiPour

updated.

updated.

11:38 am

Behrouz NematiPour

updated.

updated.

11:19 am

Behrouz NematiPour

Monday 20 Sep 2021
10:31 pm

pmontazemi  (deleted user)

We either start with Capital letter everywhere or we don't.

We either start with Capital letter everywhere or we don't.

10:30 pm

pmontazemi  (deleted user)

We either start with Capital Letter everywhere or we don't.

We either start with Capital Letter everywhere or we don't.

10:29 pm

pmontazemi  (deleted user)

Remove extra spaces after vData.

Remove extra spaces after vData.

10:22 pm

pmontazemi  (deleted user)

it is (formal) instead of it's (informal)

it is (formal) instead of it's (informal)

9:46 pm

pmontazemi  (deleted user)

RESOLVED.

RESOLVED.

9:45 pm

pmontazemi  (deleted user)

RESOLVED.

RESOLVED.

9:05 pm

Behrouz NematiPour

Fixed.

Fixed.

9:03 pm

Behrouz NematiPour

Code Review Comments

- QApplication typo

- Added a UI version of the Log Macros.

Options
    • -4
    • +4
    ./hd/adjustment/common/VCommonAdjustmentVitals.cpp
  2. … 13 more files in changeset.
8:47 pm

Behrouz NematiPour

1) This sleep helps the Bluetooth driver to load properly. 2) After many, many tests figured the Bluetooth device needs a power cycle after it is loaded and find out if giving it a 5-second delay b...

1) This sleep helps the Bluetooth driver to load properly.
2) After many, many tests figured the Bluetooth device needs a power cycle after it is loaded and find out if giving it a 5-second delay before setting the "hci" device to up, then it works fine and doesn't need the power cycle anymore. I timed the boot up and it's still under 30 seconds.
3) Yes it shouldn't break the loading of the other prototypes, since they may never even need it.

3:06 pm

pmontazemi  (deleted user)

QApplication is misspelled

QApplication is misspelled

3:06 pm

pmontazemi  (deleted user)

1) What is this sleep for? 2) How did you come up with its value? (I assume this is in seconds?) 3) Is this value consistent among all 4 prototypes and the development boards that we have?

1) What is this sleep for?
2) How did you come up with its value? (I assume this is in seconds?)
3) Is this value consistent among all 4 prototypes and the development boards that we have?

Thursday 16 Sep 2021
2:17 pm

Behrouz NematiPour

UI-DEN-9626_DEV Bluetooth Cuff Integration
UI-DEN-9626_DEV Bluetooth Cuff Integration