hdfirmware

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
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
Friday 04 Nov 2022
1:05 pm

Dong Nguyen <dnguyen@diality.com>

Merge DEN-14150

Options
    • -8
    • +0
    /firmware/App/Controllers/DGInterface.c
    • -0
    • +1
    /firmware/App/Controllers/DialInFlow.c
1:02 pm

Dara Navaei

12:09 pm

wbracken  (deleted user)

Thought we had decided that the standard would be to add the F?

Thought we had decided that the standard would be to add the F?

11:57 am

Sean Nash

Ok to calculate here (this is called before UI needs it I assume). I was expecting this to be pre-calculated on exit of treatment params mode or on entry to pre-treatment mode when we have what we ...

Ok to calculate here (this is called before UI needs it I assume). I was expecting this to be pre-calculated on exit of treatment params mode or on entry to pre-treatment mode when we have what we need to do the calculation.

11:43 am

Sean Nash

I would think this alarm would be triggered generally by driver because current mode said we shouldn't see syringe yet. Can we just set stopPump to TRUE here and nothing else?

I would think this alarm would be triggered generally by driver because current mode said we shouldn't see syringe yet.
Can we just set stopPump to TRUE here and nothing else?

11:37 am

Sean Nash

No harm in adding "F" suffix, but not necessary for assignment to a float variable - assignment statement is inherently typed. We need the "F" suffix where typing is not assured (e.g. #define where...

No harm in adding "F" suffix, but not necessary for assignment to a float variable - assignment statement is inherently typed. We need the "F" suffix where typing is not assured (e.g. #define where we don't know how it will be used).

11:33 am

Sean Nash

We probably don't need this flag at this point, but should we reset this flag on exit of retract (after treatment done)?

We probably don't need this flag at this point, but should we reset this flag on exit of retract (after treatment done)?

11:26 am

Sean Nash

I think this should probably be reset elsewhere. Init function and on exit of retract maybe?

I think this should probably be reset elsewhere. Init function and on exit of retract maybe?

11:16 am

Sean Nash

Probably could just be if ( TRUE == isSyringePumpPreLoaded() ).

Probably could just be if ( TRUE == isSyringePumpPreLoaded() ).

Thursday 03 Nov 2022
5:38 pm

Dara Navaei

Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    /firmware/App/HDCommon.h
5:37 pm

Dara Navaei

Merged DEN-14175

Options
    • -3
    • +5
    /firmware/App/Controllers/DGInterface.c
    • -1
    • +1
    /firmware/App/Controllers/DGInterface.h
    • -1
    • +1
    /firmware/App/Controllers/PresOccl.c
    • -1
    • +4
    /firmware/App/Controllers/SyringePump.c
    • -4
    • +26
    /firmware/App/Modes/ModePreTreat.c
    • -2
    • +12
    /firmware/App/Modes/Prime.c
    • -3
    • +3
    /firmware/App/Services/SystemCommMessages.c
    • -1
    • +1
    /firmware/App/Services/SystemCommMessages.h
4:00 pm

Dara Navaei

DEN-14175 replaced the V3 flag with FPGA hardware check and added the default the fixed valves PWM count in FPGA

Options
    • -2
    • +1
    /firmware/App/Controllers/Valves.c
    • -1
    • +9
    /firmware/App/Services/FPGA.c
1:42 pm

wbracken  (deleted user)

0.0F

0.0F

1:42 pm

wbracken  (deleted user)

0.0F

0.0F

1:41 pm

wbracken  (deleted user)

0.0F

0.0F

1:39 pm

Darren Cox <darren_m@cox.net>

DEN-14170 Fix TODO with new fault.

Options
    • -1
    • +1
    /firmware/App/Modes/SelfTests.c
    • -0
    • +1
    /firmware/App/Services/AlarmMgmtSWFaults.h
12:42 pm

Dara Navaei

HD-DEN-14170_SW Dev Sprint 82 Darren
HD-DEN-14170_SW Dev Sprint 82 Darren
10:23 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

10:23 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

10:20 am

wbracken  (deleted user)

Update function header.

Update function header.

10:17 am

wbracken  (deleted user)

Remove blank line

Remove blank line

9:53 am

Dara Navaei

DEN-14175 addressed code review comments

Options
    • -10
    • +0
    /firmware/App/Controllers/BloodLeak.c
9:52 am

Dara Navaei

Done.

Done.

9:51 am

Dara Navaei

This function should stay commented. I have a TODO task for this.

This function should stay commented. I have a TODO task for this.

9:50 am

Dara Navaei

Done.

Done.

9:49 am

Dara Navaei

No these should be TRUE.

No these should be TRUE.

9:45 am

Dara Navaei

Merge branch 'DEN-14175-dg_hd_dev-hd_dg_dvt-update-part-5' of ssh://dvm-linux02:7999/hd/hdfirmware into DEN-14175-dg_hd_dev-hd_dg_dvt-update-part-5

Options
9:45 am

Dara Navaei

DEN-14175 updated the reservoir switch command to send the trimmer use the previous duty cycle or estimate the duty cycle

Options
    • -3
    • +5
    /firmware/App/Controllers/DGInterface.c
    • -1
    • +1
    /firmware/App/Controllers/DGInterface.h
    • -2
    • +5
    /firmware/App/Modes/ModePostTreat.c
    • -5
    • +20
    /firmware/App/Modes/ModePreTreat.c
    • -1
    • +6
    /firmware/App/Modes/PreTreatmentRecirc.c
    • -2
    • +12
    /firmware/App/Modes/Prime.c
    • -1
    • +6
    /firmware/App/Modes/TreatmentEnd.c
    • -2
    • +6
    /firmware/App/Services/Reservoirs.c
    • -3
    • +3
    /firmware/App/Services/SystemCommMessages.c
    • -1
    • +1
    /firmware/App/Services/SystemCommMessages.h
9:04 am

Sean Nash

Add a space between "return" and "(".

Add a space between "return" and "(".

Wednesday 02 Nov 2022
5:43 pm

Dong Nguyen <dnguyen@diality.com>

Update DialInFlow.c, DGInterface.c

Options
    • -2
    • +3
    /firmware/App/Controllers/DGInterface.c