common

Clone Tools

Stats

Commits this week: 0
Total Committers: 18
Last Commit: 19 Nov 25
Commits this week: 0

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 19 Aug 2021
8:52 am

Sean Nash

DEN-8886: Addressed code review comments.

Options
    • -3
    • +3
    /Compatible.h
8:50 am

Sean Nash

Done.

Done.

8:48 am

Sean Nash

Fixed.

Fixed.

8:47 am

Sean Nash

Fixed.

Fixed.

8:46 am

Sean Nash

Common.h includes TestSupport.h.

Common.h includes TestSupport.h.

8:45 am

Sean Nash

Added.

Added.

8:43 am

Sean Nash

Changed to match your implementation.

Changed to match your implementation.

8:38 am

Sean Nash

Usually. But this function is called by AlarmMgmt (not really a h/w driver) so I kept the conversion at FPGA level to keep it low level.

Usually. But this function is called by AlarmMgmt (not really a h/w driver) so I kept the conversion at FPGA level to keep it low level.

8:36 am

Sean Nash

Fixed.

Fixed.

8:34 am

Sean Nash

Done.

Done.

8:29 am

Sean Nash

Done.

Done.

Wednesday 18 Aug 2021
9:56 pm

Dara Navaei

Please mention the usage of this include.

Please mention the usage of this include.

9:56 pm

Dara Navaei

Please add the PN and the manufacturer of the sensors.

Please add the PN and the manufacturer of the sensors.

9:49 pm

Dara Navaei

The brief does not match with what the function does.

The brief does not match with what the function does.

9:38 pm

Dara Navaei

Was it not decided to just get the FPGA counts and then do the conversion into the driver that calls the FPGA function?

Was it not decided to just get the FPGA counts and then do the conversion into the driver that calls the FPGA function?

9:34 pm

Dara Navaei

If you merge staging into your branch, the temperatures driver converts all the temperature sensors and thermistors.

If you merge staging into your branch, the temperatures driver converts all the temperature sensors and thermistors.

9:32 pm

Dara Navaei

Missing doxygen comments.

Missing doxygen comments.

4:56 pm

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Refactor re-circ mode into GenIdle mode

Options
    • -9
    • +8
    /DGDefs.h
2:57 pm

Quang Nguyen <qnguyen@diality.com>

Merge branch 'staging' into develop

Options
11:51 am

Sean Nash

DEN-8886: Removed blood sitting alarm escalation from alarm table.

Options
    • -3
    • +3
    /AlarmDefs.h
12:43 am

qnguyen  (deleted user)

Should we include "TestSupport.h" for function declaration?

Should we include "TestSupport.h" for function declaration?

12:41 am

qnguyen  (deleted user)

File name mismatch.

File name mismatch.

12:32 am

qnguyen  (deleted user)

Fix alignment.

Fix alignment.

Tuesday 17 Aug 2021
11:26 am

Sean Nash

Occlusion pressure readings are not converted to any units. All thresholds are given in counts.

Occlusion pressure readings are not converted to any units. All thresholds are given in counts.

11:25 am

Sean Nash

Done.

Done.

Monday 16 Aug 2021
4:33 pm

Dara Navaei

Please add why you are using this include.

Please add why you are using this include.

2:40 pm

Dara Navaei

Please add doxygen comments.

Please add doxygen comments.

2:38 pm

Dara Navaei

Where do you convert the FPGA data to pressure?

Where do you convert the FPGA data to pressure?

Sunday 15 Aug 2021
10:32 am

Sean Nash

DEN-8886: Added column headers at multiple points in large alarm table.

Options
    • -0
    • +11
    /AlarmDefs.h
Friday 13 Aug 2021
4:10 pm

Quang Nguyen <qnguyen@diality.com>

Enable POST state safety shutdown

Options
    • -1
    • +1
    /HDDefs.h