leahi-common

Clone Tools

Stats

Commits this week: 10
Total Committers: 16
Last Commit: 15:12
Commits this week: 10

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
Today
5 hrs ago

Vendor - TEL - Sameer Poyil

LDT-1903 Adding Air pump duty cycle out of range alarm

Options
    • -4
    • +4
    /AlarmDefs.h
6 hrs ago

Behrouz NematiPour and Nicholas Ramirez

6 hrs ago

Sean Nash

Actually, I think we should have the local variable to copy the payload into, then we should range check it (and potentially reject it if out of range), and if it is in range, only then should we a...

Actually, I think we should have the local variable to copy the payload into, then we should range check it (and potentially reject it if out of range), and if it is in range, only then should we assign new rate to your static variable.

6 hrs ago

Sean Nash

I think this can just be an else since resume is the only remaining possibility.

I think this can just be an else since resume is the only remaining possibility.

6 hrs ago

Sean Nash

Should we set resume state here? I know it's initialized to ramp state already, but in case we come back to ramp it may be safer to set it here.

Should we set resume state here? I know it's initialized to ramp state already, but in case we come back to ramp it may be safer to set it here.

6 hrs ago

Sean Nash

You moved it, but didn't change anything. The comment says it's a timer counter but the variable name suggests it is actually a flow rate. I want you to fix the comment.

You moved it, but didn't change anything. The comment says it's a timer counter but the variable name suggests it is actually a flow rate. I want you to fix the comment.

6 hrs ago

Sean Nash

I think system messages (non Dialin (test) functions should come first in this table so that they are found faster.

I think system messages (non Dialin (test) functions should come first in this table so that they are found faster.

6 hrs ago

Sean Nash

I think this would be a TD s/w fault (alarm already define). Just need a new TD s/w fault ID in AlarmMgmtSWFaults.h.

I think this would be a TD s/w fault (alarm already define). Just need a new TD s/w fault ID in AlarmMgmtSWFaults.h.

6 hrs ago

Sean Nash

This structure doesn't seem necessary - it's just a middle man. Why not just pass the #defines directly into the quadratic macro?

This structure doesn't seem necessary - it's just a middle man. Why not just pass the #defines directly into the quadratic macro?

8 hrs ago

Vendor - TEL - Sameer Poyil

Fixed

Fixed

9 hrs ago

Vendor - TEL - Sameer Poyil

removed from here

removed from here

9 hrs ago

Vendor - TEL - Sameer Poyil

fixed

fixed

Friday 27 Feb
2:56 pm

Vendor - TEL - Sameer Poyil

We need new alarm . That should be included in SRS right ? or shall i use SET_ALARM_WITH_2_U32_DATA( ALARM_ID_TD_SOFTWARE_FAULT) ?

We need new alarm . That should be included in SRS right ? or shall i use SET_ALARM_WITH_2_U32_DATA( ALARM_ID_TD_SOFTWARE_FAULT) ?

1:32 pm

Sean Nash

Add blank line after declarations.

Add blank line after declarations.

1:32 pm

Sean Nash

Alarm should be handled inside of setAirPumpState().

Alarm should be handled inside of setAirPumpState().

1:31 pm

Sean Nash

Alarm should be handled inside of setAirPumpState().

Alarm should be handled inside of setAirPumpState().

1:31 pm

Sean Nash

Alarm should be handled inside of setAirPumpState().

Alarm should be handled inside of setAirPumpState().

1:31 pm

Sean Nash

Alarm should be handled inside of setAirPumpState().

Alarm should be handled inside of setAirPumpState().

1:31 pm

Sean Nash

Alarm should be handled inside of setAirPumpState().

Alarm should be handled inside of setAirPumpState().

1:30 pm

Sean Nash

Alarm should be handled inside of setAirPumpState().

Alarm should be handled inside of setAirPumpState().

1:30 pm

Sean Nash

Add blank line after declarations.

Add blank line after declarations.

1:29 pm

Sean Nash

I think alarm should be handled inside the setAirPumpState() function.

I think alarm should be handled inside the setAirPumpState() function.

1:29 pm

Sean Nash

Should we have alarm here?

Should we have alarm here?

1:28 pm

Sean Nash

duty misspelled.

duty misspelled.

1:26 pm

Sean Nash

Can we remove inclusion of fpga here? It shouldn't be needed.

Can we remove inclusion of fpga here? It shouldn't be needed.

12:01 pm

Vendor - TEL - Varshini Nagabooshanam

Added

Added

12:01 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Removed and replaced

Removed and replaced

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Added

Added

12:00 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated