dd-firmware / firmware / App / Controllers

BalancingChamber.c

Clone Tools

Stats

Commits this week: 3
Total Committers: 11
Last Commit: 11 Aug
Commits this week: 3

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
Tuesday 18 Aug
11:39 am

Sean Nash

I'm seeing a lot of magic numbers. We should have #defines for these values.

I'm seeing a lot of magic numbers. We should have #defines for these values.

11:37 am

Sean Nash

Add doxygen comments to right.

Add doxygen comments to right.

Friday 14 Aug
10:36 am

Vendor - TEL - Sameer Poyil

The current implementation and OR and AND logic is a problem , We need a better logic here. for the time being OR is better so that it won't suck up till pressure drops to 4PSI

The current implementation and OR and AND logic is a problem , We need a better logic here. for the time being OR is better so that it won't suck up till pressure drops to 4PSI

10:25 am

Vendor - TEL - Sameer Poyil

removed

removed

10:25 am

Vendor - TEL - Sameer Poyil

updated

updated

10:02 am

Vendor - TEL - Sameer Poyil

updated

updated

10:01 am

Vendor - TEL - Sameer Poyil

updated

updated

10:00 am

Vendor - TEL - Sameer Poyil

removed

removed

Wednesday 12 Aug
3:59 pm

Sean Nash

Are we done testing? Can we remote test variables and code?

Are we done testing? Can we remote test variables and code?

3:29 pm

Vinayakam Mani

Remove Sequence step states if not being used.

Remove Sequence step states if not being used.

3:27 pm

Vinayakam Mani

Update comments ( remove start/end states)

Update comments ( remove start/end states)

3:26 pm

Vinayakam Mani

Update TD to DD.

Update TD to DD.

3:26 pm

Vinayakam Mani

Update alarm properties (DD fault, Priority eti,.)

Update alarm properties (DD fault, Priority eti,.)

3:23 pm

Vinayakam Mani

Remove the empty line.

Remove the empty line.

3:20 pm

Vinayakam Mani

Or User cancels drain request: To cancel the ongoing drain process. Could you please add the condition as well?

Or User cancels drain request: To cancel the ongoing drain process. Could you please add the condition as well?

3:14 pm

Vinayakam Mani

Remove commented code.

Remove commented code.

3:14 pm

Vinayakam Mani

Remove D34 valve close.

Remove D34 valve close.

3:13 pm

Vinayakam Mani

Add a TODO comment that D83 will be updated to new valve once new valve driver added.

Add a TODO comment that D83 will be updated to new valve once new valve driver added.

3:11 pm

Vinayakam Mani

void inside paranthesis.

void inside paranthesis.

3:01 pm

Vinayakam Mani

You can remove the above condition and do the rest of code, except D65 valve opening. Initially this state was waiting for balancing chamber fill to be completed and open D65 valve later. Now that ...

You can remove the above condition and do the rest of code, except D65 valve opening. Initially this state was waiting for balancing chamber fill to be completed and open D65 valve later. Now that this function will check the condition and skip the valves actions if not met. this is not the expected behavior.

2:19 pm

Vinayakam Mani

It's better to keep AND condition that d66 pressure must come down to decay pressure before we start vent process.

It's better to keep AND condition that d66 pressure must come down to decay pressure before we start vent process.

2:15 pm

Vinayakam Mani

Open D80, not D85.

Open D80, not D85.

2:11 pm

Vinayakam Mani

Remove the commented code.

Remove the commented code.

2:06 pm

Vinayakam Mani

Update description across the file.

Update description across the file.

1:59 pm

Vinayakam Mani

Update description.

Update description.

1:57 pm

Vinayakam Mani

update the description.

update the description.

1:46 pm

Vinayakam Mani

You need to initialize the valve states and timer etc., as similar to when you jump from idle state to fill water state. Fill water state just checks the pressure check condition.

You need to initialize the valve states and timer etc., as similar to when you jump from idle state to fill water state. Fill water state just checks the pressure check condition.

1:35 pm

Vinayakam Mani

Add TODO comment that it should be removed later.

Add TODO comment that it should be removed later.

11:25 am

Vinayakam Mani

Add comment.

Add comment.

11:20 am

Vinayakam Mani

Set default publish interval to 1 sec.

Set default publish interval to 1 sec.