td-firmware / firmware

App

Clone Tools

Stats

Commits this week: 47
Total Committers: 15
Last Commit: 04 Feb
Commits this week: 47

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
Today
9 mins ago

Vendor - TEL - Varshini Nagabooshanam

What does this function do?

What does this function do?

14 mins ago

Vendor - TEL - Varshini Nagabooshanam

Add a detail for Message Sent for the broadcast data message.

Add a detail for Message Sent for the broadcast data message.

15 mins ago

Vendor - TEL - Varshini Nagabooshanam

create local Bool

create local Bool

30 mins ago

Vendor - TEL - Varshini Nagabooshanam

Add a detail for Message Sent for the broadcast data message.

Add a detail for Message Sent for the broadcast data message.

35 mins ago

Vendor - TEL - Varshini Nagabooshanam

There should be a \b before the type of details update it for all

There should be a \b before the type of details
update it for all

36 mins ago

Vendor - TEL - Arpita Srivastava

please add a comment.

please add a comment.

37 mins ago

Vendor - TEL - Arpita Srivastava

How are we entering the complete state? If we are planning to implement this function later, can we please add a TODO?

How are we entering the complete state? If we are planning to implement this function later, can we please add a TODO?

39 mins ago

Vendor - TEL - Varshini Nagabooshanam

There should be a \b before the type of details I/p and O/p please update for all

There should be a \b before the type of details I/p and O/p please update for all

40 mins ago

Vendor - TEL - Arpita Srivastava

currentAutoEjectState should be input?

currentAutoEjectState should be input?

42 mins ago

Vendor - TEL - Arpita Srivastava

currentAutoEjectState should be input?

currentAutoEjectState should be input?

45 mins ago

Vendor - TEL - Arpita Srivastava

add autoEjectTimerCounter to outputs

add autoEjectTimerCounter to outputs

48 mins ago

Vendor - TEL - Arpita Srivastava

As per MISRA 15.7, All if, else if constructs shall be terminated with an else statement.

As per MISRA 15.7, All if, else if constructs shall be terminated with an else statement.

48 mins ago

Vendor - TEL - Arpita Srivastava

As per MISRA 15.7, All if, else if constructs shall be terminated with an else statement.

As per MISRA 15.7, All if, else if constructs shall be terminated with an else statement.

49 mins ago

Vendor - TEL - Varshini Nagabooshanam

change to ///< Tubing set install state

change to ///< Tubing set install state

54 mins ago

Vendor - TEL - Varshini Nagabooshanam

56 mins ago

Vendor - TEL - Varshini Nagabooshanam

Put it back

Put it back

57 mins ago

Vendor - TEL - Varshini Nagabooshanam

There should be a \b before the type of details (Inputs and Outputs here) for doxygen update for all

There should be a \b before the type of details (Inputs and Outputs here) for doxygen update for all

58 mins ago

Vendor - TEL - Varshini Nagabooshanam

@details \b Outputs: Please make sure to update all of them to Leahi format In this case, there should be a \b before the type of details (Inputs and Outputs here) so they will be bold when doxygen...

@details \b Outputs:
Please make sure to update all of them to Leahi format In this case, there should be a \b before the type of details (Inputs and Outputs here) so they will be bold when doxygen documentation is generated.

1 hr ago

Vendor - TEL - Varshini Nagabooshanam

@details \b Inputs:

@details \b Inputs:

1 hr ago

Vendor - TEL - Arpita Srivastava

Can we please add more comments in this function to explain the rationale behind the checks and the actions?

Can we please add more comments in this function to explain the rationale behind the checks and the actions?

1 hr ago

Michael Garthwaite

1 hr ago

Vendor - TEL - Arpita Srivastava

add a space between message and )

add a space between message and )

1 hr ago

Vendor - TEL - Arpita Srivastava

Change // to /// for doxygen to catch it.

Change // to /// for doxygen to catch it.

1 hr ago

Vendor - TEL - Arpita Srivastava

Can we remove the parameter and switch directly over currentPostTreatmentState?

Can we remove the parameter and switch directly over currentPostTreatmentState?

2 hrs ago

Vendor - TEL - Arpita Srivastava

The public function name signalGoToInstallState is not sufficiently descriptive. Would renaming it to requestTubeSetInstall() be appropriate?

The public function name signalGoToInstallState is not sufficiently descriptive. Would renaming it to requestTubeSetInstall() be appropriate?

2 hrs ago

Sean Nash

I prefer you create a local BOOL result so you can have a simpler return statement.

I prefer you create a local BOOL result so you can have a simpler return statement.

2 hrs ago

Sean Nash

Should go to your new complete state here.

Should go to your new complete state here.

2 hrs ago

Vendor - TEL - Arpita Srivastava

The number looks incorrect. This was an issue in the staging branch, which is fixed now. Can you please pull latest staging and update the values of your enums?

The number looks incorrect. This was an issue in the staging branch, which is fixed now. Can you please pull latest staging and update the values of your enums?

Monday 04 May
7:01 pm

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-4248_(TD) RO Loop Testing / Demo: TD pinch valves to be in the correct state at standby and treatment
LEAHI-TD-FIRMWARE-LDT-4248_(TD) RO Loop Testing / Demo: TD pinch valves to be in the correct state at standby and treatment
4:30 pm

Sean Nash

check alarm status instead of switch state.

check alarm status instead of switch state.