Activity dd-firmware/

Tree
Accordion

Stats

Commits this week: 41

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

Lines of code count not available

Commit Activity

last updated a few seconds ago

Constraints

Constraints: committers

Committer:

Log Comment:

Constraints: files

File Extension:

File Name:

Constraints: dates

Start Date:

Start Date:

End Date:

End Date:

Today

3 hrs ago

Michael Garthwaite

More
333baf3
changed 2 files
LDT-2004-v2-conductivity-sensors LEAHI-DD-FIRMWARE-LDT-2004-4

LDT-2004: warning reductions for teensy driver

- -2
- +4
/firmware/App/Drivers/ConductivitySensors.h
- History
- Source
- Diff
/firmware/App/.../ConductivitySensors.h
- -109
- +5
/firmware/App/Drivers/ConductivityTeensy.c
- History
- Source
- Diff
/firmware/App/.../ConductivityTeensy.c

4 hrs ago

Michael Garthwaite

added

added

4 hrs ago

Michael Garthwaite

added

added

4 hrs ago

Michael Garthwaite

More
3b97a99
changed 1 file
LDT-3372-v1-conductivity-fixes LEAHI-DD-FIRMWARE-LDT-3372-2

LDT-3372: code review comments.

- -13
- +10
/firmware/App/Drivers/ConductivityTeensy.c
- History
- Source
- Diff
/firmware/App/.../ConductivityTeensy.c

4 hrs ago

Michael Garthwaite

fixed.

fixed.

4 hrs ago

Michael Garthwaite

then we return false as expected

then we return false as expected

4 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.

4 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 )

4 hrs ago

Michael Garthwaite

removed. thanks!

removed. thanks!

4 hrs ago

Michael Garthwaite

parameter check happens in getTeensyCondId

parameter check happens in getTeensyCondId

5 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

5 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 )

5 hrs ago

Dara Navaei

More
50951be
changed 1 file
develop

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

- -1
- +1
/firmware/App/DDCommon.h
- History
- Source
- Diff
/firmware/App/DDCommon.h

5 hrs ago

Michael Garthwaite

More
fdd4492
changed 0 files
develop

Merge branch 'LDT-3372-v1-conductivity-fixes' into develop

5 hrs ago

Michael Garthwaite

More
37e25a9
changed 1 file
LDT-3372-v1-conductivity-fixes, develop LEAHI-DD-FIRMWARE-LDT-3372-2

LDT-3372: fixes to conductivity calculation.

- -5
- +5
/firmware/App/Drivers/ConductivityTeensy.c
- History
- Source
- Diff
/firmware/App/.../ConductivityTeensy.c

6 hrs ago

Dara Navaei

More
24efc54
changed 1 file
develop

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

- -1
- +1
/firmware/App/DDCommon.h
- History
- Source
- Diff
/firmware/App/DDCommon.h

6 hrs ago

Vinayakam Mani

More
4aa373d
changed 1 file
develop

LDT-1477 merge into develop

- -4
- +20
/firmware/App/Controllers/BalancingChamber.c
- History
- Source
- Diff
/firmware/App/.../BalancingChamber.c

6 hrs ago

Vinayakam Mani

More
f554e39
changed 2 files
LDT-1477-ultrafiltration---dd---04-dev--, develop LEAHI-DD-FIRMWARE-LDT-1477-1

LDT-1477 trimmer heater target update

- -4
- +20
/firmware/App/Controllers/BalancingChamber.c
- History
- Source
- Diff
/firmware/App/.../BalancingChamber.c
- -2
- +16
/firmware/App/Modes/ModeGenDialysate.c
- History
- Source
- Diff
/firmware/App/Modes/ModeGenDialysate.c

8 hrs ago

Vendor - TEL - Sameer Poyil

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

its unsigned integer , no need to check -ve value

its unsigned integer , no need to check -ve value

Friday 27 Feb

11:28 am

Vendor - TEL - Arpita Srivastava

More
c71a1a9
changed 9 files
LDT-1195-dd-is-balancing-chamber-switching LEAHI-DD-FIRMWARE-LDT-1195-2

LDT-1195: Updated balancing chamber switching request flags names and definitions.

- -89
- +62
/firmware/App/Controllers/BalancingChamber.c
- History
- Source
- Diff
/firmware/App/.../BalancingChamber.c
- -3
- +2
/firmware/App/Controllers/BalancingChamber.h
- History
- Source
- Diff
/firmware/App/.../BalancingChamber.h
- -0
- +2
/firmware/App/Modes/ModeFault.c
- History
- Source
- Diff
/firmware/App/Modes/ModeFault.c
- -17
- +22
/firmware/App/Modes/ModeGenDialysate.c
- History
- Source
- Diff
/firmware/App/Modes/ModeGenDialysate.c
- -0
- +2
/firmware/App/Modes/ModeInitPOST.c
- History
- Source
- Diff
/firmware/App/Modes/ModeInitPOST.c
- -1
- +2
/firmware/App/Modes/ModePostGenDialysate.c
- History
- Source
- Diff
/firmware/App/.../ModePostGenDialysate.c
- -1
- +2
/firmware/App/Modes/ModePreGenDialysate.c
- History
- Source
- Diff
/firmware/App/Modes/ModePreGenDialysate.c
- -0
- +2
/firmware/App/Modes/ModeService.c
- History
- Source
- Diff
/firmware/App/Modes/ModeService.c
- -5
- +10
/firmware/App/Modes/ModeStandby.c
- History
- Source
- Diff
/firmware/App/Modes/ModeStandby.c

7:36 am

Vendor - TEL - Sameer Poyil

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

is this function required?

is this function required?

7:32 am

Vendor - TEL - Sameer Poyil

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

remove all commented code

remove all commented code

7:31 am

Vendor - TEL - Sameer Poyil

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

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

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

what if user send -ve values ?

what if user send -ve values ?

7:24 am

Vendor - TEL - Sameer Poyil

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

Where is len used ?

Where is len used ?

7:08 am

Vendor - TEL - Sameer Poyil

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

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

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

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

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

need math formula as documentation

need math formula as documentation

7:00 am

Vendor - TEL - Sameer Poyil

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

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

More
Re: LEAHI-DD-FIRMWARE-LDT-3372-2
commented on review

Need a detailed name for variable - apply for all other

Need a detailed name for variable - apply for all other

Atlassian Fisheye Git analysis with Crucible code review. (Version:4.8.14 Build:20240111091859 2024-01-11 09:20) - Administration - Page generated 2026-03-02 16:27 -0800