leahi-fwcommon

PIControllers.c

Clone Tools

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 03 Feb
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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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

Friday 05 Dec 2025
3:01 pm

Sean Nash

I'm confused as to why the available alarm that this alarm replaced is in a different row within this table. That seems wrong.

I'm confused as to why the available alarm that this alarm replaced is in a different row within this table. That seems wrong.

1:26 pm

Dara Navaei

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

Options
    • -4
    • +4
    /PIControllers.c
  2. … 28 more files in changeset.
Thursday 04 Dec 2025
10:18 am

Raghu Kallala

Handled in FPOperationModes and respective states of PreGen have it.

Handled in FPOperationModes and respective states of PreGen have it.

10:17 am

Raghu Kallala

Checked with Eliza if they need it for alarm persistence time. Not needed at this time, we can add them in future

Checked with Eliza if they need it for alarm persistence time. Not needed at this time, we can add them in future

10:15 am

Raghu Kallala

Fixed. Thanks

Fixed. Thanks

10:14 am

Raghu Kallala

Fixed.

Fixed.

10:11 am

Raghu Kallala

Fixed.

Fixed.

10:10 am

Raghu Kallala

Fixed. thanks

Fixed. thanks

9:00 am

Sean Nash

I don't think the number of samples is what you want as 3rd parameter. You want number of bytes so 4 x number of samples so you should pass "sizeof( roRRSamples )".

I don't think the number of samples is what you want as 3rd parameter. You want number of bytes so 4 x number of samples so you should pass "sizeof( roRRSamples )".

8:57 am

Sean Nash

Put space between if and ( for all conditions in this function.

Put space between if and ( for all conditions in this function.

8:55 am

Sean Nash

Put first condition in () too.

Put first condition in () too.

Wednesday 03 Dec 2025
5:58 pm

Vendor - TEL - Sameer Poyil

Small letter used

Small letter used

10:38 am

Raghu Kallala

Resolved.

Resolved.

10:34 am

Raghu Kallala

Resolved. Thanks

Resolved. Thanks

8:43 am

Raghu Kallala

We might need to provide a possibility to override the persistence time on alarm. To be clarified.

We might need to provide a possibility to override the persistence time on alarm. To be clarified.

8:38 am

Raghu Kallala

Published in the respective modules

Published in the respective modules

8:37 am

Raghu Kallala

Done in individual units publish functions.

Done in individual units publish functions.