dialin

Clone Tools

Stats

Commits this week: 0
Total Committers: 21
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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 13 Dec 2022
1:29 pm

Dara Navaei

DEN-14307 changed the message ID of the DG crc NV record override due to conflict

Options
    • -1
    • +3
    /dialin/common/msg_ids.py
1:21 pm

Dara Navaei

DEN-14307 updated the DG tests

Options
    • -77
    • +17
    /tests/dg_tests.py
Monday 12 Dec 2022
3:42 pm

Dara Navaei

Bamboo Commit: generated the code change report

Options
    • -5
    • +44
    /tools/Code_Report.csv
3:40 pm

Dara Navaei

Merged staging with the latest changes

Options
    • -2
    • +5
    /dialin/common/msg_ids.py
    • -27
    • +204
    /dialin/hd/battery.py
    • -0
    • +2
    /dialin/hd/hemodialysis_device.py
3:21 pm

Behrouz NematiPour

Friday 09 Dec 2022
10:39 am

Dara Navaei

Bamboo Commit: generated the code change report

Options
    • -1
    • +48
    /tools/Code_Report.csv
10:37 am

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -1
    • +1
    /dialin/common/msg_ids.py
    • -6
    • +6
    /dialin/hd/battery.py
    • -2
    • +2
    /dialin/hd/hemodialysis_device.py
10:33 am

Dara Navaei

5:11 am

Behrouz NematiPour

Thursday 08 Dec 2022
2:28 pm

Sean Nash and Michael Garthwaite

2:28 pm

Sean Nash

RESOLVED.

RESOLVED.

2:26 pm

Dara Navaei

DEN-14001 addressed a code review comments

Options
    • -0
    • +1
    /dialin/common/msg_ids.py
2:25 pm

Dara Navaei

Done.

Done.

2:19 pm

Sean Nash

May not be needed, but can we just add 0xB8 too for completeness?

May not be needed, but can we just add 0xB8 too for completeness?

1:01 pm

Dara Navaei

DEN-14001 merged staging

Options
    • -11
    • +20
    /dialin/common/msg_ids.py
    • -6
    • +6
    /dialin/hd/hemodialysis_device.py
11:35 am

Sean Nash

Looking at unpack functionality a little more, I believe HH will parse a 32-bit byte array as 2 unsigned shorts. I think it's kind of working because byte array is little endian and we're only look...

Looking at unpack functionality a little more, I believe HH will parse a 32-bit byte array as 2 unsigned shorts. I think it's kind of working because byte array is little endian and we're only looking at the first value (index 0) parsed.
Firmware is sending everything as U32. So we should be using "I" for unsigned int.

11:31 am

Sean Nash

RESOLVED.

RESOLVED.

11:29 am

Dara Navaei

Please look at the green lines at the bottom of these comments

Please look at the green lines at the bottom of these comments

11:26 am

Michael Garthwaite

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

11:22 am

Sean Nash

first get function still missing header.

first get function still missing header.

11:21 am

Sean Nash

RESOLVED.

RESOLVED.

11:21 am

Sean Nash

RESOLVED.

RESOLVED.

11:20 am

Sean Nash

RESOLVED.

RESOLVED.

11:20 am

Sean Nash

RESOLVED.

RESOLVED.

11:20 am

Sean Nash

RESOLVED.

RESOLVED.

11:20 am

Sean Nash

Must have worked. I think just one H is all that's needed for unsigned short. Not sure why type "int" is being parsed as unsigned short though.

Must have worked. I think just one H is all that's needed for unsigned short. Not sure why type "int" is being parsed as unsigned short though.

11:16 am

Sean Nash

RESOLVED.

RESOLVED.

11:07 am

Michael Garthwaite

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

11:07 am

Michael Garthwaite

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

11:04 am

Dara Navaei

DEN-14001 addressed the code review comments

Options
    • -32
    • +25
    /dialin/hd/battery.py