leahi-dialin / leahi_dialin / fp / modules

valves.py

Clone Tools

Stats

Commits this week: 0
Total Committers: 6
Last Commit: 30 Sep 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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 30 Mar
9:39 am

Sean Nash

Thursday 26 Mar
12:16 pm

Dara Navaei

LEAHI-DIALIN-LDT-3218_Update IOFP permeate target flow rate to 800 mL/min (and flow control as needed)
LEAHI-DIALIN-LDT-3218_Update IOFP permeate target flow rate to 800 mL/min (and flow control as needed)
Monday 23 Mar
9:13 am

Zoltan Miskolci

Wednesday 18 Mar
1:27 pm

Michael Garthwaite and Raghu Kallala

Tuesday 17 Mar
7:05 am

Raghu Kallala

update params

update params

Monday 16 Mar
12:06 pm

Dara Navaei

LEAHI-DIALIN-LDT-3629_FP and DD valve_states_enum are one short of the full valve list
LEAHI-DIALIN-LDT-3629_FP and DD valve_states_enum are one short of the full valve list
10:06 am

Michael Garthwaite

yeah its obsolete. This is an old branch that recently got synced with latest staging. It looks like the CR doesn't recognize the correct changes from staging. There should only be 1 file change ( ...

yeah its obsolete. This is an old branch that recently got synced with latest staging. It looks like the CR doesn't recognize the correct changes from staging. There should only be 1 file change ( conductivity sensors ).

Thursday 12 Mar
2:32 pm

Zoltan Miskolci

2:12 pm

Michael Garthwaite and Raghu Kallala

11:50 am

Zoltan Miskolci

Fixed

Fixed

11:48 am

Zoltan Miskolci

added Todo comment

added Todo comment

11:47 am

Zoltan Miskolci

They were added while debugging fp_test_configs and why it's never being updated. Removed.

They were added while debugging fp_test_configs and why it's never being updated.
Removed.

10:59 am

Michael Garthwaite

tip: use the logger and set the statement to error level and you can see it in runtime

tip: use the logger and set the statement to error level and you can see it in runtime

9:51 am

Raghu Kallala

pressure limit will be unsigned int not signed int with LDT-473..beta CR Add a TODO to update once the 473 is ready to merge

pressure limit will be unsigned int not signed int with LDT-473..beta CR
Add a TODO to update once the 473 is ready to merge

9:45 am

Raghu Kallala

Are these print statements add for debugging? If not, is this allowed

Are these print statements add for debugging?
If not, is this allowed

9:36 am

Raghu Kallala

typo, PUMP_ON_STATE

typo, PUMP_ON_STATE

Tuesday 10 Mar
11:48 am

Dara Navaei

LEAHI-DIALIN-LDT-3629_FP and DD valve_states_enum are one short of the full valve list
LEAHI-DIALIN-LDT-3629_FP and DD valve_states_enum are one short of the full valve list
Thursday 05 Mar
10:46 am

Zoltan Miskolci

10:25 am

Raghu Kallala

Wednesday 04 Mar
12:46 pm

Dara Navaei

12:46 pm

Dara Navaei

Tuesday 03 Mar
4:58 pm

Michael Garthwaite

3:28 pm

Zoltan Miskolci

removed

removed

9:13 am

Michael Garthwaite

Remove this extra line

Remove this extra line

Friday 20 Feb
10:46 am

Dara Navaei

LEAHI-DIALIN-LDT-3557_[Documantation][Enhancement] Add documentation header to dialin variables
LEAHI-DIALIN-LDT-3557_[Documantation][Enhancement] Add documentation header to dialin variables
Monday 09 Feb
4:49 pm

Zoltan Miskolci

1:47 pm

Sean Nash

9:16 am

Michael Garthwaite

This class member is a way for dialin to present all the valve states in a list like so: >>> fp = FP() >>> fp.valves.valve_states_enum [0, 0, 0, 0, 0, 0, 0] It is organized in the order of the e...

This class member is a way for dialin to present all the valve states in a list like so:

>>> fp = FP()
>>> fp.valves.valve_states_enum
[0, 0, 0, 0, 0, 0, 0]


It is organized in the order of the enum that was used to iterate. This is purely for presentation / Quality of life feature for the dialin user. We can remove it if we know users are no longer using it for monitoring

Friday 06 Feb
2:14 pm

Sean Nash

Can you review this with Michael G and find out if this is even needed? Can we remove? If not, what is it doing and why?

Can you review this with Michael G and find out if this is even needed? Can we remove? If not, what is it doing and why?

9:47 am

Zoltan Miskolci

I see it as a list, don't know why it is named enum or what the original purpose of it was. It was never updated, so I added the update part at least, so it will at least reflect the current states...

I see it as a list, don't know why it is named enum or what the original purpose of it was.
It was never updated, so I added the update part at least, so it will at least reflect the current states instead of always stating zeros.