dgfirmware / firmware / App

Controllers

Clone Tools

Stats

Commits this week: 0
Total Committers: 16
Last Commit: 19 Nov 25
Commits this week: 0

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
Tuesday 28 Mar 2023
5:27 pm

Bill Bracken <bill@criticalsoftwaresolutions.com>

Merge branch 'staging' into DEN-14646-fpga-alarm-properties-and-timing-3

Options
    • -7
    • +15
    ./ConductivitySensors.c
    • -2
    • +7
    ./Heaters.c
    • -3
    • +3
    ./LoadCell.c
    • -1
    • +2
    ./TemperatureSensors.c
    • -3
    • +3
    ./Valves.c
5:19 pm

Dara Navaei

5:08 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

5:07 pm

Dara Navaei

Done

Done

5:03 pm

Sean Nash

I did. I still see a call to sendAckResponseMsg() and I don't think it should be here.

I did. I still see a call to sendAckResponseMsg() and I don't think it should be here.

5:00 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

4:59 pm

Dara Navaei

DEN-15014 addressed the code review comment

Options
    • -1
    • +1
    ./ROPump.c
4:59 pm

Dara Navaei

Check the diff of the last two commits.

Check the diff of the last two commits.

4:58 pm

Sean Nash

What I meant is that we don't need to send anything other than response.

What I meant is that we don't need to send anything other than response.

4:58 pm

Dara Navaei

Done

Done

4:57 pm

Sean Nash

Remove TODO?

Remove TODO?

4:56 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

4:55 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

4:54 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

4:54 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

4:52 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

4:51 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

4:50 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

4:50 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

4:48 pm

Dara Navaei

DEN-15014 addressed the code review comments

Options
    • -1
    • +1
    ./ConductivitySensors.c
    • -1
    • +1
    ./ROPump.c
  3. … 4 more files in changeset.
4:47 pm

Dara Navaei

Done

Done

4:43 pm

Dara Navaei

Changed it to TRUE/FALSE.

Changed it to TRUE/FALSE.

4:43 pm

Dara Navaei

Updated the code.

Updated the code.

4:42 pm

Dara Navaei

Done

Done

4:17 pm

Dara Navaei

Done

Done

4:17 pm

Dara Navaei

Done

Done

4:15 pm

Dara Navaei

Done

Done

4:13 pm

Dara Navaei

Done

Done

4:10 pm

Sean Nash

Prefer to say RO only != TRUE so that any other value than 1 will cause RR alarm to be enforced.

Prefer to say RO only != TRUE so that any other value than 1 will cause RR alarm to be enforced.

4:08 pm

Sean Nash

Needs param so caller can set it to FALSE too?

Needs param so caller can set it to FALSE too?