leahi-common

Clone Tools

Stats

Commits this week: 33
Total Committers: 19
Last Commit: 14:13
Commits this week: 33

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
3 hrs ago

Vendor - TEL - Jashwant Gantyada

resolved merge conflicts and merged LDT-3963

Options
    • -8
    • +8
    /AlarmDefs.h
    • -3
    • +17
    /MsgDefs.h
    • -2
    • +3
    /TDDefs.h
3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Merge branch 'staging' of bitbucket.org:diality-cloud/leahi-common into LDT-3215-measured-venous-pressure-decreases

Options
3 hrs ago

Sean Nash and Vinayakam Mani

4 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Thats for error register so not required now

Thats for error register so not required now

5 hrs ago

Vendor - TEL - Varshini Nagabooshanam

You’re right. I should also check the busy status before transitioning to the GET_DATA state. I’ll update the measurement completion logic to first verify the busy bit is cleared, then validate the...

You’re right. I should also check the busy status before transitioning to the GET_DATA state. I’ll update the measurement completion logic to first verify the busy bit is cleared, then validate the BP data response code before moving to BP_DRIVER_GET_DATA_STATE.

5 hrs ago

Vendor - TEL - Jashwant Gantyada

Merge branch 'staging' into feature/LDT-3963-dialysate-flow-rate---fw-implem

Options
Thursday 21 May
11:16 pm

Vendor - TEL - Vijay Pamula

LDT:3970 Merge Latest From Staging

Options
    • -4
    • +31
    /MsgDefs.h
    • -22
    • +26
    /TDDefs.h
10:58 pm

Vendor - TEL - Vijay Pamula

LDT:3970 Updated Edit Treatment Parameters

Options
    • -6
    • +6
    /MsgDefs.h
    • -1
    • +1
    /TDDefs.h
10:46 pm

Sean Nash

Why do you have {} in the cases?

Why do you have {} in the cases?

10:45 pm

Sean Nash

You are getting bpResults, but not doing anything with them. Should you pass as parameter to checkBloodPressureReading() function?

You are getting bpResults, but not doing anything with them. Should you pass as parameter to checkBloodPressureReading() function?

10:43 pm

Sean Nash

Where are these get functions defined? I can't find them.

Where are these get functions defined? I can't find them.

8:45 pm

Dara Navaei

LDT-3259 merged staging

Options
    • -13
    • +66
    /AlarmDefs.h
    • -4
    • +31
    /MsgDefs.h
    • -22
    • +26
    /TDDefs.h
8:41 pm

Vendor - TEL - Praneeth Bunne

LDT-1886: Resolved merge conflicts

Options
    • -2
    • +28
    /MsgDefs.h
    • -2
    • +3
    /TDDefs.h
5:11 pm

Vendor - TEL - Varshini Nagabooshanam

Merge branch 'staging' of bitbucket.org:diality-cloud/leahi-common into LDT-4033-patient-vitals-fw-implementation

Options
5:10 pm

Vendor - TEL - Varshini Nagabooshanam

LDT-3215-measured-venous-pressure-decreases

Options
    • -55
    • +3
    /AlarmDefs.h
4:27 pm

Vendor - TEL - Arpita Srivastava

3:36 pm

Vendor - TEL - Jashwant Gantyada

LDT-3963 addressed code review comments

Options
    • -6
    • +6
    /AlarmDefs.h
3:24 pm

Vendor - TEL - Jashwant Gantyada

restored

restored

3:24 pm

Sean Nash

Not sure why Crucible is showing it this way. Let's just re-review these diffs.

Not sure why Crucible is showing it this way. Let's just re-review these diffs.

3:22 pm

Dara Navaei

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

Options
    • -2
    • +2
    /MsgDefs.h
    • -2
    • +2
    /TDDefs.h
3:18 pm

Sean Nash

Should be #ifndef

Should be #ifndef

3:13 pm

Sean Nash

Should be #ifndef I think.

Should be #ifndef I think.

3:06 pm

Sean Nash

Can we make these low priority and 999 rank?

Can we make these low priority and 999 rank?

3:06 pm

Vendor - TEL - Jashwant Gantyada and Sean Nash

1:33 pm

Vendor - TEL - Jashwant Gantyada

LDT-3963 addressed code review comments

Options
    • -8
    • +8
    /AlarmDefs.h
    • -2
    • +2
    /DDDefs.h
12:11 pm

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

12:02 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

11:41 am

Vendor - TEL - Varshini Nagabooshanam

Added comments

Options
    • -7
    • +60
    /AlarmDefs.h
    • -3
    • +15
    /MsgDefs.h
    • -22
    • +25
    /TDDefs.h
11:28 am

Sean Nash

Do we not need the module error alarm?

Do we not need the module error alarm?

11:27 am

Sean Nash

This is not an atomic assignment, so we have to consider thread safety. If this driver is running from General Task (I believe it is since ModeTreatment's exec will be calling the BPModule's exec w...

This is not an atomic assignment, so we have to consider thread safety.
If this driver is running from General Task (I believe it is since ModeTreatment's exec will be calling the BPModule's exec which calls this driver's exec) and if the caller to this function will also be running from General Task (I believe it is since BPModule controller is the caller), then there is no thread safety issue and we should mention in the function header brief that this function should only be called from within the General Task.
If another task is involved (e.g. Priority Task is calling this function), this function could be interrupting the get data state while it is populating bpResults with new results and the caller would get a mix of old and new results (we would not want that).