leahi-fwcommon

PIControllers.c

Clone Tools

Stats

Commits this week: 1
Total Committers: 5
Last Commit: 13 Mar
Commits this week: 1

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 13 Mar
9:33 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:31 am

Sean Nash

Should we create a structure for these moving average filters and make an array of them based on enum of temp sensors? It looks kind of messy having separate handling for each sensor - can we conso...

Should we create a structure for these moving average filters and make an array of them based on enum of temp sensors?
It looks kind of messy having separate handling for each sensor - can we consolidate some of this code?

9:28 am

Sean Nash

Should we maybe just have a constant array of sample counts based on enum of temp sensors?

Should we maybe just have a constant array of sample counts based on enum of temp sensors?

9:24 am

Sean Nash

Should min and max be properties of the controller instead of global (assumes all controllers will want same min/max)? Controller properties already include a min and max field - can we use those?

Should min and max be properties of the controller instead of global (assumes all controllers will want same min/max)?
Controller properties already include a min and max field - can we use those?

9:16 am

Sean Nash

Remove blank line.

Remove blank line.

Thursday 12 Mar
3:43 pm

Vinayakam Mani

Done.

Done.

3:42 pm

Vinayakam Mani

Done.

Done.

Wednesday 25 Feb
12:49 pm

Vendor - TEL - Sameer Poyil

mis aligned ?

mis aligned ?

12:48 pm

Vendor - TEL - Sameer Poyil

mis aligned ?

mis aligned ?

Monday 23 Feb
6:29 pm

Vendor - TEL - Sameer Poyil

need a brief description about this feature and how it is implemented/logic/math etc - also comment on the previous logic

need a brief description about this feature and how it is implemented/logic/math etc - also comment on the previous logic

6:27 pm

Vendor - TEL - Sameer Poyil

its a reverse logic - is it OK to make positive instead of not defined ?

its a reverse logic - is it OK to make positive instead of not defined ?

Friday 13 Feb
3:47 pm

Sean Nash

Looks like we are doing a lot of the dialysate temp control in Gen Dialysate mode. Do we not do any heating in PreGen mode? Even if all dialysate heating is done in this mode, should we consider cr...

Looks like we are doing a lot of the dialysate temp control in Gen Dialysate mode. Do we not do any heating in PreGen mode?
Even if all dialysate heating is done in this mode, should we consider creating a service controller (like BC controller) to manage this complex temperature control?

3:38 pm

Sean Nash

Resolve merge conflicts.

Resolve merge conflicts.

Wednesday 11 Feb
1:49 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3218_Update IOFP permeate target flow rate to 800 mL/min (and flow control as needed)
LEAHI-DD-FIRMWARE-LDT-3218_Update IOFP permeate target flow rate to 800 mL/min (and flow control as needed)
Tuesday 10 Feb
5:03 pm

Sean Nash

Add blank line between declarations and rest of code.

Add blank line between declarations and rest of code.

5:02 pm

Sean Nash

Add comment to right listing which math functions we're using.

Add comment to right listing which math functions we're using.

12:04 pm

Dara Navaei

LEAHI-DD-FIRMWARE-LDT-3352_DD - revisit D3 level control logic
LEAHI-DD-FIRMWARE-LDT-3352_DD - revisit D3 level control logic
Friday 09 Jan
10:08 am

Dara Navaei

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

Options
    • -1
    • +1
    /PIControllers.c
  2. … 28 more files in changeset.
Friday 19 Dec 2025
11:26 am

Michael Garthwaite

Thursday 18 Dec 2025
3:11 pm

Sean Nash

3:11 pm

Sean Nash

That suggests that our coefficient(s) are too aggressive (in situations where it looks unstable). Not directly due to max step size, so we shouldn't reduce it for this reason. I believe 25 will neg...

That suggests that our coefficient(s) are too aggressive (in situations where it looks unstable). Not directly due to max step size, so we shouldn't reduce it for this reason. I believe 25 will negatively impact our responsiveness in situations where our error is larger than 25 mL/min.
So instability could be due to:
1) coefficient(s) are too strong or
2) may need more than one set of coefficients (e.g. coefficients need to change according to Qd or state or ... because the relationship between output and feedback changes)

11:00 am

Michael Garthwaite

increasing it to 50 would cause instability in our testing. 25 seems to keep it stable at the moment

increasing it to 50 would cause instability in our testing. 25 seems to keep it stable at the moment

Tuesday 09 Dec 2025
11:25 am

Raghu Kallala

11:15 am

Dara Navaei

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

Options
    • -2
    • +2
    /PIControllers.c
10:59 am

Raghu Kallala

LDT-2936 merge into staging

Options
    • -5
    • +5
    /PIControllers.c
10:43 am

Sean Nash , Vinayakam Mani and Michael Garthwaite

10:29 am

Raghu Kallala

Resolved

Resolved

10:16 am

Sean Nash

Change comment from 3 to 5 minutes to match code.

Change comment from 3 to 5 minutes to match code.

10:14 am

Sean Nash

I don't see fix

I don't see fix

10:03 am

Raghu Kallala

Fixed the ordering

Fixed the ordering