dd-firmware / firmware / App / Controllers

ConcentratePumps.c

Clone Tools

Stats

Commits this week: 13
Total Committers: 8
Last Commit: 19 Mar
Commits this week: 13

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
Wednesday 18 Mar
6:58 pm

Dara Navaei

Merged staging and resolved the conflicts

Options
    • -28
    • +19
    ./ConcentratePumps.c
  2. … 10 more files in changeset.
5:37 pm

Dara Navaei

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

Options
    • -2
    • +2
    ./ConcentratePumps.c
  2. … 5 more files in changeset.
4:58 pm

Dara Navaei

Merged LDT-3750

Options
    • -1
    • +1
    ./ConcentratePumps.c
  2. … 1 more file in changeset.
4:47 pm

Dara Navaei

LDT-3750 updated the balancing chamber and concentrate pumps

Options
    • -1
    • +1
    ./ConcentratePumps.c
  2. … 1 more file in changeset.
3:37 pm

Dara Navaei

2:40 pm

Sean Nash

Add TODO comment to restore later.

Add TODO comment to restore later.

Tuesday 17 Mar
6:20 pm

Vendor - TEL - Jashwant Gantyada

LDT- 3628 resolved merge conflicts

Options
    • -3
    • +3
    ./ConcentratePumps.c
6:16 pm

Vendor - TEL - Jashwant Gantyada

LDT-3628 updated code

Options
    • -1
    • +1
    ./ConcentratePumps.c
6:03 pm

Vendor - TEL - Jashwant Gantyada

Merge branch 'LDT-3628-request-to-change-the-target-re' into develop

Options
    • -1
    • +2
    ./ConcentratePumps.c
6:02 pm

Vendor - TEL - Jashwant Gantyada

LDT 3628 fixes for develop

Options
    • -1
    • +2
    ./ConcentratePumps.c
2:46 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3750_DD_Staging_Rebase
LEAHI-DD-FIRMWARE-LDT-3750_DD_Staging_Rebase
11:41 am

Vendor - TEL - Jashwant Gantyada

updated.

updated.

11:41 am

Vendor - TEL - Jashwant Gantyada

updated.

updated.

11:41 am

Vendor - TEL - Jashwant Gantyada

done.

done.

10:21 am

Vendor - TEL - Jashwant Gantyada

Merge branch 'LDT-3628-request-to-change-the-target-re' into develop

Options
    • -6
    • +6
    ./ConcentratePumps.c
  2. … 1 more file in changeset.
10:19 am

Vendor - TEL - Jashwant Gantyada

LDT-3628 aadressed code review comments

Options
    • -2
    • +2
    ./ConcentratePumps.c
  2. … 1 more file in changeset.
10:03 am

Vinayakam Mani

4 bytes reg.

4 bytes reg.

10:02 am

Vinayakam Mani

4 byte reg.

4 byte reg.

9:52 am

Vinayakam Mani

Remove typecast.

Remove typecast.

9:41 am

Vendor - TEL - Jashwant Gantyada

LDT-3628 fixed fpga mapping issues

Options
    • -4
    • +4
    ./ConcentratePumps.c
  2. … 2 more files in changeset.
Monday 16 Mar
5:21 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3628_Request to change the target rev count for piston pumps from 2 byte to 4 byte register
LEAHI-DD-FIRMWARE-LDT-3628_Request to change the target rev count for piston pumps from 2 byte to 4 byte register
11:27 am

Vendor - TEL - Jashwant Gantyada

Merge branch 'LDT-3628-request-to-change-the-target-re' into develop

Options
    • -6
    • +6
    ./ConcentratePumps.c
  2. … 2 more files in changeset.
11:26 am

Vendor - TEL - Jashwant Gantyada

LDT-3628 changed piston pump target rev counts from 2 to 4 bytes

Options
    • -6
    • +6
    ./ConcentratePumps.c
  2. … 3 more files in changeset.
Wednesday 04 Mar
2:05 pm

Sean Nash

Looked better before - now misspelled.

Looked better before - now misspelled.

2:04 pm

Sean Nash

0.0F

0.0F

2:04 pm

Sean Nash

indent looks 1 space off.

indent looks 1 space off.

2:01 pm

Sean Nash

If we include math, I like to see comment explaining what we're doing with it. Why remove it?

If we include math, I like to see comment explaining what we're doing with it. Why remove it?

2:00 pm

Sean Nash

Declare at top of scope, assign down here if needed.

Declare at top of scope, assign down here if needed.

1:59 pm

Sean Nash

If there is only one target temperature, why are we passing in a heater ID to function? This seems weird to me having both heaters share a target.

If there is only one target temperature, why are we passing in a heater ID to function? This seems weird to me having both heaters share a target.

1:57 pm

Sean Nash

Remove blank line.

Remove blank line.