dd-firmware / firmware / App / Services

Messaging.c

Clone Tools

Stats

Commits this week: 2
Total Committers: 14
Last Commit: 20 Aug
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
Thursday 20 Aug
1:33 pm

Dara Navaei

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

Options
    • -2
    • +2
    ./Messaging.c
  2. … 6 more files in changeset.
1:32 pm

Vendor - TEL - Suresh Dharnala

11:50 am

Michael Garthwaite and Vinayakam Mani

Wednesday 19 Aug
1:40 pm

Michael Garthwaite

flip the logic such that featured is the IF and not the ELSE to avoid having a blank branch statement

flip the logic such that featured is the IF and not the ELSE to avoid having a blank branch statement

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.

Monday 17 Aug
4:59 pm

Sean Nash

3:37 pm

suresh

LDT-4049:removed disinfection related code

Options
    • -3
    • +0
    ./Messaging.c
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
4:06 pm

suresh

LDT-4049: Added msg hanlers

Options
    • -0
    • +3
    ./Messaging.c
  2. … 3 more files in changeset.
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.