dd-firmware / firmware / App / Controllers

ConcentratePumps.h

Clone Tools

Stats

Commits this week: 2
Total Committers: 2
Last Commit: 30 Jul 24
Commits this week: 2

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
Today
5 mins ago

Sean Nash

Remove blank line.

Remove blank line.

11 mins ago

Sean Nash

Should we add _TARGET to these names for clarity?

Should we add _TARGET to these names for clarity?

14 mins ago

Sean Nash

Is the count really a float?

Is the count really a float?

14 mins ago

Sean Nash

Do we need an override for the RPM feedback?

Do we need an override for the RPM feedback?

17 mins ago

Sean Nash

This is a maximum per treatment?

This is a maximum per treatment?

1 hr ago

Sean Nash

I'm not seeing fixes in this file.

I'm not seeing fixes in this file.

1 hr ago

Sean Nash

Remove extra blank line.

Remove extra blank line.

1 hr ago

Michael Garthwaite

fixed. thanks!

fixed. thanks!

1 hr ago

Michael Garthwaite

fixed. thanks!

fixed. thanks!

1 hr ago

Michael Garthwaite

fixed. thanks!

fixed. thanks!

1 hr ago

Michael Garthwaite

fixed. thanks!

fixed. thanks!

1 hr ago

Michael Garthwaite

moved the sets into a separate function from init

moved the sets into a separate function from init

Monday 08 Dec
1:02 pm

Sean Nash

Move up with other controllers.

Move up with other controllers.

Thursday 16 Oct
9:52 am

Sean Nash

I only see these configurations being set here in initialize function. Is that all we need? Is this configuration what we want for these valves in all states? I would recommend we combine these 3 ...

I only see these configurations being set here in initialize function. Is that all we need? Is this configuration what we want for these valves in all states?

I would recommend we combine these 3 fpga functions into one single function that takes all 3 params and updates all 3 registers.

9:35 am

Sean Nash

I think these should be initialized to zero in case there are unused bits that won't be set below.

I think these should be initialized to zero in case there are unused bits that won't be set below.

9:30 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:26 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

Tuesday 07 Oct
2:15 pm

jpaguio  (deleted user)

2:15 pm

jpaguio  (deleted user)

LDT-2198 was merged into this branch after being merged into staging_alpha. Branch will be killed and relevant code changes were reviewed appropriately.

LDT-2198 was merged into this branch after being merged into staging_alpha. Branch will be killed and relevant code changes were reviewed appropriately.

9:46 am

Sean Nash

Monday 06 Oct
11:01 am

Sean Nash and Vinayakam Mani

11:00 am

Raghu Kallala , Michael Garthwaite and Vinayakam Mani

Friday 03 Oct
2:45 pm

Vinayakam Mani

Done.

Done.

Wednesday 01 Oct
10:47 am

Vinayakam Mani

Tuesday 30 Sep
1:33 pm

Vinayakam Mani

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

11:45 am

jpaguio  (deleted user)

Done

Done

11:45 am

jpaguio  (deleted user)

Done

Done

11:45 am

jpaguio  (deleted user)

Done

Done

11:44 am

jpaguio  (deleted user)

Done

Done

11:44 am

jpaguio  (deleted user)

Done

Done