dd-firmware / firmware

App

Clone Tools

Stats

Commits this week: 20
Total Committers: 17
Last Commit: 11:57
Commits this week: 20

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
Today
50 mins ago

Vinayakam Mani

51 mins ago

Vinayakam Mani

This needs TD source update and also dialin update, keep informed system team about this change as this would impact DD demo running.

This needs TD source update and also dialin update, keep informed system team about this change as this would impact DD demo running.

54 mins ago

Vinayakam Mani

Please add HDF Pump speed register and add a monitor in the substitution pump controller to compare against the target speed and if the difference is out of range, trigger an alarm.

Please add HDF Pump speed register and add a monitor in the substitution pump controller to compare against the target speed and if the difference is out of range, trigger an alarm.

1 hr ago

Vinayakam Mani

Substitution pump

Substitution pump

4 hrs ago

Sean Nash

Use ternary to explicitly assign TRUE or FALSE.

Use ternary to explicitly assign TRUE or FALSE.

4 hrs ago

Sean Nash

Use ternary to explicitly assign TRUE or FALSE.

Use ternary to explicitly assign TRUE or FALSE.

5 hrs ago

Vinayakam Mani

Missing comment.

Missing comment.

5 hrs ago

Vinayakam Mani

update comment. next one as well.

update comment. next one as well.

5 hrs ago

Vinayakam Mani

blank line after the banner and maintain alphabetical order of include files.

blank line after the banner and maintain alphabetical order of include files.

7 hrs ago

Michael Garthwaite

LDT-4001: previous CR comments. updates to TD interface and messaging

Options
    • -34
    • +56
    ./Controllers/SubstitutionPump.c
    • -9
    • +8
    ./Controllers/SubstitutionPump.h
    • -0
    • +4
    ./Services/Messaging.c
    • -1
    • +1
    ./Services/TDInterface.c
8 hrs ago

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-4001_HDF - FW Implementation - 2/4:
LEAHI-DD-FIRMWARE-LDT-4001_HDF - FW Implementation - 2/4:
8 hrs ago

Vinayakam Mani

Done.

Done.

8 hrs ago

Michael Garthwaite

LDT-4001: fixes and merges from previous working branch.

Options
    • -1
    • +1
    ./Controllers/RinsePump.c
    • -0
    • +73
    ./Controllers/SubstitutionPump.h
    • -6
    • +0
    ./Monitors/Conductivity.c
    • -1
    • +3
    ./Services/FpgaDD.c
    • -0
    • +4
    ./Services/FpgaDD.h
    • -0
    • +2
    ./Services/TDInterface.c
9 hrs ago

Michael Garthwaite

LDT-4000: changed beta 2.0 test config to beta 1.9 test config

Options
    • -7
    • +7
    ./Controllers/Heaters.c
    • -9
    • +4
    ./Controllers/RinsePump.c
    • -4
    • +4
    ./Monitors/Conductivity.c
    • -24
    • +22
    ./Monitors/Level.c
    • -12
    • +12
    ./Monitors/Temperature.c
    • -27
    • +93
    ./Services/FpgaDD.c
9 hrs ago

Vendor - TEL - Jashwant Gantyada

9 hrs ago

Vendor - TEL - Jashwant Gantyada

I think we also need this to be uncommented, as we were running into an alarm because of this.

I think we also need this to be uncommented, as we were running into an alarm because of this.

9 hrs ago

Vinayakam Mani

Added new field for d1 filtered temperature and restored the pressure sensor temperature.

Added new field for d1 filtered temperature and restored the pressure sensor temperature.

9 hrs ago

Vinayakam Mani

The latest code from staging has an update and fixes this comment. It's not showing here, since the file was already reviewed.

The latest code from staging has an update and fixes this comment. It's not showing here, since the file was already reviewed.

9 hrs ago

Vinayakam Mani

as discussed, these were outcome of the test results. we will review these gains with latest controller.

as discussed, these were outcome of the test results. we will review these gains with latest controller.

9 hrs ago

Vinayakam Mani

Done.

Done.

9 hrs ago

Vinayakam Mani

Done.

Done.

Thursday 28 May
1:47 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3975_Water Treatment and Degassing - FW Iteration - 4/4:
LEAHI-DD-FIRMWARE-LDT-3975_Water Treatment and Degassing - FW Iteration - 4/4:
Wednesday 27 May
4:24 pm

Dara Navaei

3:43 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./DDCommon.h
3:43 pm

Dara Navaei

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

Options
    • -0
    • +16
    ./Modes/ModeUpdate.c
    • -0
    • +16
    ./Modes/ModeUpdate.h
    • -2
    • +2
    ./Modes/OperationModes.c
    • -2
    • +2
    ./Services/AlarmMgmtSWFaults.h
    • -2
    • +2
    ./Services/Messaging.c
    • -2
    • +2
    ./Services/Messaging.h
    • -2
    • +2
    ./Services/SystemCommDD.c
    • -2
    • +2
    ./Services/SystemCommDD.h
3:38 pm

Vendor - TEL - Jashwant Gantyada and Sean Nash

2:46 pm

Sean Nash

On DD f/w side, this should just be a simple pump controller. States should be Off and On (control to commanded Qsub). There is no flow sensor for feedback control, so assuming this is open loop si...

On DD f/w side, this should just be a simple pump controller. States should be Off and On (control to commanded Qsub). There is no flow sensor for feedback control, so assuming this is open loop similar to UF pump.
GenDialysate mode will have an HDF sub-state that signals this controller from Off to On and would also call separate controller for heater and separate (Switches) monitor for door sensors.

With all of that in mind, I think this unit should have a simpler name like SubPump.

2:39 pm

Sean Nash

I think there should be some state monitoring for potential alarm(s) in here too. Similar to BP door on TD f/w side where the current mode/state can signal this unit that it expects specific switch...

I think there should be some state monitoring for potential alarm(s) in here too. Similar to BP door on TD f/w side where the current mode/state can signal this unit that it expects specific switch state(s) (e.g. while in HDF state, we would expect the cap to be open and stowed and we would want to alarm if that was not the case).

2:36 pm

Sean Nash

Remove extra blank line.

Remove extra blank line.

2:33 pm

Dara Navaei

LDT-3272 addressed the code review comments

Options
    • -1
    • +1
    ./Services/Messaging.c
    • -16
    • +0
    ./Services/SystemCommDD.h
  3. … 2 more files in changeset.