Dashboard

Welcome Message

Welcome to Atlassian Crucible with Fisheye .

To report a bug or suggest a feature please raise an issue in our public Jira.

Customize this welcome note.

Support

You are running Crucible 4.8.14 with Fisheye 4.8.14 (20240111091859 2024-01-11 09:20)

If you are experiencing any issues, please contact your administrator. You may also want to browse the documentation or search the support forums. Still stuck? You can start a support ticket at support.atlassian.com or raise a bug or feature request at jira.atlassian.com.

Customize the support message.

Today
4 mins ago

Vendor - TEL - Arpita Srivastava

LDT-3505: Added test config for syringe pump

Options
    • -1
    • +1
    /leahi_dialin/common/test_config_defs.py
8 mins ago

Vendor - TEL - Arpita Srivastava

We can rename priorState to previousInstallState. Also, it should be declared as a static variable to be able to retain previousState value when the control goes out of the function.

We can rename priorState to previousInstallState.
Also, it should be declared as a static variable to be able to retain previousState value when the control goes out of the function.

11 mins ago

Sean Nash and Vendor - TEL - Arpita Srivastava

12 mins ago

Vendor - TEL - Praneeth Bunne

LDT-1886: Add send event

Options
    • -0
    • +7
    /firmware/App/Services/StateServices/TubeSetAutoEject.c
    • -0
    • +7
    /firmware/App/Services/StateServices/TubeSetInstall.c
38 mins ago

Sean Nash

SW fault here?

SW fault here?

38 mins ago

Sean Nash

Do we need this state if all it's doing is going to IDLE?

Do we need this state if all it's doing is going to IDLE?

39 mins ago

Sean Nash

Might have to wait for module to not be busy before getting the data.

Might have to wait for module to not be busy before getting the data.

42 mins ago

Sean Nash

I think we may need to give a get data command here.

I think we may need to give a get data command here.

42 mins ago

Sean Nash

Look for request flag to give measure command and transition to measure state.

Look for request flag to give measure command and transition to measure state.

43 mins ago

Sean Nash

Write private handle functions for each state and call them in each case.

Write private handle functions for each state and call them in each case.

44 mins ago

Sean Nash

Should we also check whether bpMeasurementReady is TRUE?

Should we also check whether bpMeasurementReady is TRUE?

45 mins ago

Sean Nash

Same.

Same.

46 mins ago

Sean Nash

Same. Set a request flag to TRUE (for Peds this time) and let the state machine do the rest.

Same. Set a request flag to TRUE (for Peds this time) and let the state machine do the rest.

46 mins ago

Sean Nash

Don't change state from outside signal. We should just set a requestBPMeasurement flag to TRUE in this function (if current state is IDLE) and then when the state machine runs next, the IDLE state ...

Don't change state from outside signal. We should just set a requestBPMeasurement flag to TRUE in this function (if current state is IDLE) and then when the state machine runs next, the IDLE state should look for this flag to be TRUE and transition to MEASURE state (by doing all of these things you have here in this function).

50 mins ago

Sean Nash

Change comment to "Number of BP driver states."

Change comment to "Number of BP driver states."

52 mins ago

Vendor - TEL - Varshini Nagabooshanam

LDT-4033-patient-vitals-fw-implementation

Options
    • -1
    • +1
    /firmware/App/Controllers/BPModule.c
    • -1
    • +0
    /firmware/App/Controllers/BPModule.h
    • -9
    • +0
    /firmware/App/Drivers/BPDriver.h
1 hr ago

Vendor - TEL - Arpita Srivastava

LDT-3505: Addressed review comments

Options
    • -32
    • +90
    /firmware/App/Controllers/SyringePump.c
    • -1
    • +1
    /firmware/App/Services/FpgaTD.c
1 hr ago

Vendor - TEL - Varshini Nagabooshanam

LDT-4033-patient-vitals-fw-implementation

Options
    • -0
    • +1
    /firmware/App/Controllers/BPModule.c
    • -0
    • +10
    /firmware/App/Drivers/BPDriver.c
1 hr ago

Vendor - TEL - Varshini Nagabooshanam

Not really, I can change this to private

Not really, I can change this to private

1 hr ago

Sean Nash

Add a 5th enum for NUM_OF_BP_DRIVER_STATES. And, does this need to be public?

Add a 5th enum for NUM_OF_BP_DRIVER_STATES.
And, does this need to be public?

1 hr ago

Raghu Kallala

1 hr ago

Vendor - TEL - Jashwant Gantyada

align indentation, same for line 338 as well.

align indentation, same for line 338 as well.

1 hr ago

Vendor - TEL - Arpita Srivastava

Yes. Otherwise, we will have to hard code the calibration record for the time being.

Yes.
Otherwise, we will have to hard code the calibration record for the time being.

1 hr ago

Vendor - TEL - Arpita Srivastava

The function getSoftwareConfigStatus() is also part of NVM module. Moreover, we are planning to remove this function when NVM will be implemented. So, we will have to perform a cleanup and remove a...

The function getSoftwareConfigStatus() is also part of NVM module.
Moreover, we are planning to remove this function when NVM will be implemented.
So, we will have to perform a cleanup and remove all similar calls that have been currently commented out in other files as well like AirTrap.c, BloodFlow.c, etc.

2 hrs ago

Vendor - TEL - Varshini Nagabooshanam

New module

Options
    • -0
    • +199
    /firmware/App/Controllers/BPModule.c
    • -0
    • +45
    /firmware/App/Controllers/BPModule.h
    • -0
    • +216
    /firmware/App/Drivers/BPDriver.c
    • -0
    • +68
    /firmware/App/Drivers/BPDriver.h
2 hrs ago

Sean Nash

if syrPumpDACControl is U16 (2 bytes), then nibpCtl should be at 42.

if syrPumpDACControl is U16 (2 bytes), then nibpCtl should be at 42.

2 hrs ago

Sean Nash

Remove extra blank line.

Remove extra blank line.

2 hrs ago

Vendor - TEL - Varshini Nagabooshanam

LDT-4033-patient-vitals-fw-implementation

Options
    • -0
    • +24
    /AlarmDefs.h
2 hrs ago

Vendor - TEL - Varshini Nagabooshanam

LDT-4033-patient-vitals-fw-implementation

Options
    • -114
    • +27
    /firmware/App/Services/FpgaTD.c
    • -40
    • +16
    /firmware/App/Services/FpgaTD.h
3 hrs ago

Sean Nash

Is this the right time to zero this volume?

Is this the right time to zero this volume?