dd-firmware

Clone Tools

Stats

Commits this week: 41
Total Committers: 15
Last Commit: 13:13
Commits this week: 41

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
8 hrs ago

Raghu Kallala

9 hrs ago

Michael Garthwaite

LDT-3514: wrapped ALARM_ID_FP_POWER_BUDGET_OUT_OF_RANGE around test config TEST_CONFIG_FP_DISABLE_RO_CURRENT_ALARM.

Options
    • -1
    • +5
    /firmware/App/Monitors/WaterQualityMonitor.c
9 hrs ago

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3514_Dialin config request to disable IOFP current monitoring
LEAHI-DD-FIRMWARE-LDT-3514_Dialin config request to disable IOFP current monitoring
10 hrs ago

Michael Garthwaite

added

added

10 hrs ago

Michael Garthwaite

added

added

10 hrs ago

Michael Garthwaite

fixed.

fixed.

10 hrs ago

Michael Garthwaite

then we return false as expected

then we return false as expected

10 hrs ago

Michael Garthwaite

These are correct variable names based on their coefficient names in calculations of conductivity.

These are correct variable names based on their coefficient names in calculations of conductivity.

10 hrs ago

Michael Garthwaite

Can you clarify? this is allowing us to switch conductivity calculations based on the needs of the dialin user. ( systems team testing for accuracy )

Can you clarify? this is allowing us to switch conductivity calculations based on the needs of the dialin user. ( systems team testing for accuracy )

10 hrs ago

Michael Garthwaite

removed. thanks!

removed. thanks!

10 hrs ago

Michael Garthwaite

parameter check happens in getTeensyCondId

parameter check happens in getTeensyCondId

11 hrs ago

Michael Garthwaite

Yes as it is a way for us to identify if the teensy is not responding to us correctly. This along with no LED activity on the board helps us identify teensy failures

Yes as it is a way for us to identify if the teensy is not responding to us correctly. This along with no LED activity on the board helps us identify teensy failures

11 hrs ago

Michael Garthwaite

Not for this situation since its an array of structs and looks like a 2d array. Please see static const FP_OP_MODE_T MODE_TRANSITION_TABLE in FPOperationModes.c static const DD_OP_MODE_T MODE_TRAN...

Not for this situation since its an array of structs and looks like a 2d array. Please see

static const FP_OP_MODE_T MODE_TRANSITION_TABLE in FPOperationModes.c
static const DD_OP_MODE_T MODE_TRANSITION_TABLE in OperationModes.c
static PI_CONTROLLER_T piControllers in PIControllers.c ( FWCommon repo )

14 hrs ago

Vendor - TEL - Sameer Poyil

its unsigned integer , no need to check -ve value

its unsigned integer , no need to check -ve value

Friday 27 Feb
7:36 am

Vendor - TEL - Sameer Poyil

is this function required?

is this function required?

7:32 am

Vendor - TEL - Sameer Poyil

remove all commented code

remove all commented code

7:31 am

Vendor - TEL - Sameer Poyil

good to have doxygen comments on right hand side

good to have doxygen comments on right hand side

7:26 am

Vendor - TEL - Sameer Poyil

what if user send -ve values ?

what if user send -ve values ?

7:24 am

Vendor - TEL - Sameer Poyil

Where is len used ?

Where is len used ?

7:08 am

Vendor - TEL - Sameer Poyil

its a public function , input parameter check required, apply to all public functions

its a public function , input parameter check required, apply to all public functions

7:07 am

Vendor - TEL - Sameer Poyil

need documentation for this two approach, what is the merits ? when to use it ?

need documentation for this two approach, what is the merits ? when to use it ?

7:06 am

Vendor - TEL - Sameer Poyil

need math formula as documentation

need math formula as documentation

7:00 am

Vendor - TEL - Sameer Poyil

Need mathematical formula as documentation and need to explain each term

Need mathematical formula as documentation and need to explain each term

6:56 am

Vendor - TEL - Sameer Poyil

Need a detailed name for variable - apply for all other

Need a detailed name for variable - apply for all other

Thursday 26 Feb
11:46 am

Dara Navaei

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
Wednesday 25 Feb
10:03 pm

Vendor - TEL - Sameer Poyil

3:31 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-1195_DD: Request for a variable for balancing chamber switching to be active (T/F)
LEAHI-DD-FIRMWARE-LDT-1195_DD: Request for a variable for balancing chamber switching to be active (T/F)
1:47 pm

Sean Nash

DO NOT REVIEW - temporary branch to support new Diener 1000 pump changes. Will delete branch and CR when done.

DO NOT REVIEW - temporary branch to support new Diener 1000 pump changes. Will delete branch and CR when done.

1:35 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3103_implement closed loop control for d48 pump speed
LEAHI-DD-FIRMWARE-LDT-3103_implement closed loop control for d48 pump speed
12:49 pm

Vendor - TEL - Sameer Poyil

mis aligned ?

mis aligned ?