Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LDT-3344 addressed code review comments

LEAHI-DD-FIRMWARE-LDT-3372_Add ability to override the Teensie conductivity and temperature values directly
LEAHI-DD-FIRMWARE-LDT-3372_Add ability to override the Teensie conductivity and temperature values directly
LDT-2004: updates to v2 driver. changed data types for cal coefficents to f32. Implemented calculated resistance, temp, and conductivity based on raw data from sensor.

    • -21
    • +129
    ./Drivers/ConductivitySensors.c
Bamboo Commit: Updated DDCommon.h with build versions from Bamboo.

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

For B1.0/B1.9 - Where Diener silencer 2000 pumps, this macro needs to be enabled.

For B1.0/B1.9 - Where Diener silencer 2000 pumps, this macro needs to be enabled.

There are recent changes done on FPGA register in staging for B1.0/B1.9. please merge the latest changes here.

There are recent changes done on FPGA register in staging for B1.0/B1.9. please merge the latest changes here.

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

Add valve details per HDD (may be 2.0 valves. as we remove beta 1/1.9 in sometime)

Add valve details per HDD (may be 2.0 valves. as we remove beta 1/1.9 in sometime)

add new valves.

add new valves.

Can we follow the same terminology as DD, starting a variable with 'fpga...'. for all IOFP variables?

Can we follow the same terminology as DD, starting a variable with 'fpga...'. for all IOFP variables?

typo - used.

typo - used.

Add as HDF hallsensor. start the comment with capital letter R.

Add as HDF hallsensor. start the comment with capital letter R.

#define 0U across the file.

#define 0U across the file.

Instead of generic function, recommend adding a specific function something similar to "getACSwitchStatus' or 'getLeaskSensorStatus' and return the values ( ex: leak sensor Enum states?). Do we nee...

Instead of generic function, recommend adding a specific function something similar to "getACSwitchStatus' or 'getLeaskSensorStatus' and return the values ( ex: leak sensor Enum states?). Do we need to add the application code as well now, say if leak is found, trigger an alarm?

either typecast pwmInCount as U08 while passing to FPGA or declare the variable size as U08 instead of U32.

either typecast pwmInCount as U08 while passing to FPGA or declare the variable size as U08 instead of U32.

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

resolved merge conflicts

Add a variable to convert PWM count to target speed for monitor function.

Add a variable to convert PWM count to target speed for monitor function.

LDT-3344 WIP for rinsepump implementations

Please add 'execRinsePumpMonitor' which checks the commanded vs measured speed tolerance check and trigger alarm if deviates beyond the limit. You may move this function into monitor.

Please add 'execRinsePumpMonitor' which checks the commanded vs measured speed tolerance check and trigger alarm if deviates beyond the limit. You may move this function into monitor.

LDT-2030 addressed code review comments

Use #define macros for magic numbers.

Use #define macros for magic numbers.

LDT-3344 fixed the issue of conductive level sensor response

LDT-3344 fixed the issue of conductive level sensor response

LDT-2030 addressed code review comments