dd-firmware

firmware

Clone Tools

Stats

Commits this week: 22
Total Committers: 17
Last Commit: 12:10
Commits this week: 22

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
Wednesday 27 May
2:33 pm

Dara Navaei

LDT-3272 addressed the code review comments

Options
    • -1
    • +1
    ./App/Services/Messaging.c
    • -16
    • +0
    ./App/Services/SystemCommDD.h
    • -16
    • +16
    ./DD.dil
    • -10
    • +10
    ./source/can.c
2:31 pm

Sean Nash

Where is call to HDF controller?

Where is call to HDF controller?

2:31 pm

Sean Nash

Monitors should come before mode mgmt above.

Monitors should come before mode mgmt above.

2:30 pm

Sean Nash

What about HemoDiaFiltration.h?

What about HemoDiaFiltration.h?

2:29 pm

Sean Nash

What about initHDF?

What about initHDF?

2:28 pm

Sean Nash

Where is HemoDiaFiltration.h?

Where is HemoDiaFiltration.h?

2:24 pm

Sean Nash

Why does this file name have pound signs pre/post?

Why does this file name have pound signs pre/post?

2:14 pm

Sean Nash

This macro is already defined in Common.h.

This macro is already defined in Common.h.

2:14 pm

Sean Nash

Align all "=" or none of them.

Align all "=" or none of them.

2:13 pm

Sean Nash

DD HDF control data published should just be state and set rate. TD HDF data would have volume, etc...

DD HDF control data published should just be state and set rate. TD HDF data would have volume, etc...

2:11 pm

Sean Nash

This looks like pump control function. I think TD f/w should be doing this when setting Qsub before sending to DD f/w. Again, I think DD f/w should just be controlling pump to the Qsub rate given b...

This looks like pump control function. I think TD f/w should be doing this when setting Qsub before sending to DD f/w. Again, I think DD f/w should just be controlling pump to the Qsub rate given by TD f/w.
HDF volume should be calculated by TD f/w by integrating commanded Qsub over time.

2:08 pm

Sean Nash

Remove blank line.

Remove blank line.

2:08 pm

Sean Nash

Refactor so that only one return statement at end of function. This is in our coding standards.

Refactor so that only one return statement at end of function. This is in our coding standards.

2:08 pm

Dara Navaei

This file has been updated by HALCoGen automatically.

This file has been updated by HALCoGen automatically.

2:07 pm

Dara Navaei

This file has been updated by HALCoGen automatically.

This file has been updated by HALCoGen automatically.

2:07 pm

Sean Nash

I think TD should be handling all of these rules before setting Qsub and sending to DD.

I think TD should be handling all of these rules before setting Qsub and sending to DD.

2:06 pm

Dara Navaei

This file has been updated by HALCoGen automatically.

This file has been updated by HALCoGen automatically.

2:06 pm

Dara Navaei

This file has been updated by HALCoGen automatically.

This file has been updated by HALCoGen automatically.

2:05 pm

Dara Navaei

Done

Done

2:05 pm

Dara Navaei

Done

Done

1:54 pm

Sean Nash

Generally, DD shouldn't need TMP. TD is calculating TMP. TD is using TMP to calculate Qsub. DD is just getting Qsub from TD and controlling the pump to that rate.

Generally, DD shouldn't need TMP. TD is calculating TMP. TD is using TMP to calculate Qsub. DD is just getting Qsub from TD and controlling the pump to that rate.

1:53 pm

Sean Nash

Indent declarations.

Indent declarations.

1:50 pm

Sean Nash

How can we help you with this?

How can we help you with this?

1:49 pm

Sean Nash

Doesn't match file name. Why do we have a redundant file name with ~ suffix for your new units? If these are backup files from your editor, please do not add these to the git repo (delete them befo...

Doesn't match file name. Why do we have a redundant file name with ~ suffix for your new units? If these are backup files from your editor, please do not add these to the git repo (delete them before commit).

1:48 pm

Sean Nash

DD is not calculating the rate I think. TD is sending target HDF rate to DD. So "calculated" should be "requested" or "commanded".

DD is not calculating the rate I think. TD is sending target HDF rate to DD. So "calculated" should be "requested" or "commanded".

1:46 pm

Dara Navaei

Done

Done

1:41 pm

Sean Nash

Add space before )

Add space before )

1:40 pm

Sean Nash

Add blank line before return statement.

Add blank line before return statement.

1:38 pm

Sean Nash

Align doxygen comments to right.

Align doxygen comments to right.

1:38 pm

Sean Nash

Add blank line before banner

Add blank line before banner