hdfirmware

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
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
Friday 02 Dec 2022
5:05 pm

Michael Garthwaite

i think there's a conflict between staging and develop. fixed for staging merge. thanks!

i think there's a conflict between staging and develop. fixed for staging merge. thanks!

5:04 pm

Michael Garthwaite

originally wanted to change setDialInPumpTargetFlowRate() to have the same conditions as testSetDialInPWM but we can integrate it later pending Darren's results.

originally wanted to change setDialInPumpTargetFlowRate() to have the same conditions as testSetDialInPWM but we can integrate it later pending Darren's results.

5:02 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

5:02 pm

Michael Garthwaite

Fixed. moved to testSetDialInPumpWithPWM(). Thanks!

Fixed. moved to testSetDialInPumpWithPWM(). Thanks!

5:02 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

5:02 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

5:02 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

5:01 pm

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

5:01 pm

Michael Garthwaite

Fixed. This condition requires that the Pump is flagged off. that same condition is also used below. we will always be true in the first condition below if we satisfy this condition. If we try to c...

Fixed. This condition requires that the Pump is flagged off. that same condition is also used below. we will always be true in the first condition below if we satisfy this condition. If we try to change direction when the pump is on, both this condition and the one below will fail.

4:38 pm

Sean Nash

Created a #define for magic number. Because running pumps in open loop mode (because we're priming and not necessarily any fluid in line to measure flow with) and pumps have different methods of de...

Created a #define for magic number.
Because running pumps in open loop mode (because we're priming and not necessarily any fluid in line to measure flow with) and pumps have different methods of determining PWM duty cycle from a given flow rate, DPo pumps target flow rate needed to be set lower to match the duty cycle with DPi pump.

4:24 pm

Sean Nash

Where did this enum name change take place? There is no common in this code review.

Where did this enum name change take place? There is no common in this code review.

4:22 pm

Sean Nash

remove "," after pwm.

remove "," after pwm.

4:22 pm

Sean Nash

Remove blank line.

Remove blank line.

4:21 pm

Sean Nash

Should this be set to true even if result below is false? Maybe set to true in testSetDialInPumpWithPWM() function after all of the conditions are found to be met?

Should this be set to true even if result below is false? Maybe set to true in testSetDialInPumpWithPWM() function after all of the conditions are found to be met?

4:19 pm

Sean Nash

Remove blank line.

Remove blank line.

4:19 pm

Sean Nash

First part of condition should have () around it too. Do we want to set direction here even if we fail to satisfy condition below?

First part of condition should have () around it too.
Do we want to set direction here even if we fail to satisfy condition below?

4:15 pm

Sean Nash

Keep a blank line between declaration and code.

Keep a blank line between declaration and code.

4:14 pm

Sean Nash

Not clear on what this refactoring of condition changed - looks the same.

Not clear on what this refactoring of condition changed - looks the same.

11:49 am

wbracken  (deleted user)

parameter0?

parameter0?

10:55 am

Dara Navaei

Please add comment.

Please add comment.

10:52 am

Dara Navaei

HD-DEN-14344_SW S85 MG Dpi Pwmflow Fix
HD-DEN-14344_SW S85 MG Dpi Pwmflow Fix
9:37 am

jtaylor  (deleted user)

It seems strange to add a magic number here, from a defined constant. I'd assume (guess) the relation between 300->225 is maintaining a delta P along the flow path (or across the membrane?), but it...

It seems strange to add a magic number here, from a defined constant.
I'd assume (guess) the relation between 300->225 is maintaining a delta P along the flow path (or across the membrane?), but it might be worth spelling out, at the top with the defines, or as a macro function of the input flow rate.

Tuesday 29 Nov 2022
10:31 am

jtaylor  (deleted user)

8:43 am

Michael Garthwaite

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

Monday 28 Nov 2022
4:17 pm

wbracken  (deleted user)

Alignment throughout the file. However, this file is auto-generated by HalCoGen.

Alignment throughout the file. However, this file is auto-generated by HalCoGen.

4:05 pm

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

2:02 pm

Michael Garthwaite

HalCoGen is currently editing het.h, het.c, sys_link.cmd, HD.dil, and sys_selftest.c in this branch. Any and all the changes within those files are autogenerated from HaloCoGen.

HalCoGen is currently editing het.h, het.c, sys_link.cmd, HD.dil, and sys_selftest.c in this branch. Any and all the changes within those files are autogenerated from HaloCoGen.

1:53 pm

wbracken  (deleted user)

Is this an output file from HalCoGen? If so seems like one shouldn't need to modify this file directly.

Is this an output file from HalCoGen? If so seems like one shouldn't need to modify this file directly.

12:33 pm

dcox  (deleted user)

Updated to reset valve position and flag.

Updated to reset valve position and flag.

12:32 pm

dcox  (deleted user)

Added define. There is no harm in closing valve if already closed. TODO comment below is a place holder for checking if closed too long. More testing by systems needed to determine if this is neces...

Added define. There is no harm in closing valve if already closed. TODO comment below is a place holder for checking if closed too long. More testing by systems needed to determine if this is necessary.