hdfirmware / firmware / App / Modes

SelfTests.c

Clone Tools

Stats

Commits this week: 0
Total Committers: 10
Last Commit: 19 Nov
Commits this week: 0

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
Monday 15 May 2023
10:33 am

Michael Garthwaite

DIAL-251: code review comments.

Options
    • -1
    • +0
    ./SelfTests.c
10:32 am

Michael Garthwaite

yes we should. fixed to have these calls in their respective transition functions.

yes we should. fixed to have these calls in their respective transition functions.

10:32 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:30 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:29 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:29 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:29 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:29 am

Michael Garthwaite

unneeded code from other event usage. Fixed. Thanks!

unneeded code from other event usage. Fixed. Thanks!

10:29 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:29 am

Michael Garthwaite

Fixed. Thanks!

Fixed. Thanks!

10:20 am

Dara Navaei

HD-DEN-15245_Heparin Empty Syringe Alarm Notification Not Displayed TO User IF Another Alarm IS
HD-DEN-15245_Heparin Empty Syringe Alarm Notification Not Displayed TO User IF Another Alarm IS
10:13 am

Michael Garthwaite

going to keep the old style of events for now so we dont break previous V&V testing.

going to keep the old style of events for now so we dont break previous V&V testing.

10:08 am

Michael Garthwaite

Blood prime and treatment start sub modes dont have a substate. The call here will cover the treatment start sub mode. Blood prime will be covered in the transition to blood prime

Blood prime and treatment start sub modes dont have a substate. The call here will cover the treatment start sub mode. Blood prime will be covered in the transition to blood prime

9:57 am

Sean Nash

DEN-15245: Fixed issue with syringe occlusion check potentially overriding a previously set stop pump flag.

Options
    • -1
    • +1
    ./SelfTests.c
  2. … 2 more files in changeset.
9:56 am

Michael Garthwaite

I want to keep the old sub state change events for dialin compatibility. I believe other tests use the sub state change event

I want to keep the old sub state change events for dialin compatibility. I believe other tests use the sub state change event

Sunday 14 May 2023
6:32 pm

Sean Nash

This is a 4th level state - not currently supported I think.

This is a 4th level state - not currently supported I think.

6:30 pm

Sean Nash

Should this be in transition function?

Should this be in transition function?

6:29 pm

Sean Nash

Is there any submode of treatment mode that does not have substates?

Is there any submode of treatment mode that does not have substates?

6:28 pm

Sean Nash

Not many submodes in pre-tx that don't have substates - maybe cartridge install. Should call this when entering a submode with no substates.

Not many submodes in pre-tx that don't have substates - maybe cartridge install. Should call this when entering a submode with no substates.

5:44 pm

Sean Nash

Post treatment starts in drain reservoirs submode which does have a substate. Once we leave drain reservoirs submode, then we should call this function with no substate for remainder of post treatm...

Post treatment starts in drain reservoirs submode which does have a substate. Once we leave drain reservoirs submode, then we should call this function with no substate for remainder of post treatment submodes.

5:41 pm

Sean Nash

In consumable self-test, you removed the old send substate change event. Why keep this one?

In consumable self-test, you removed the old send substate change event. Why keep this one?

5:40 pm

Sean Nash

This is a 4th level state. Currently not supported I think.

This is a 4th level state. Currently not supported I think.

5:40 pm

Sean Nash

This is a 4th level state. Currently not supported I think.

This is a 4th level state. Currently not supported I think.

5:39 pm

Sean Nash

Not clear to me which of the older events are still being supported. What's the plan?

Not clear to me which of the older events are still being supported. What's the plan?

5:38 pm

Sean Nash

Remove extra "}". Should be 2 blank lines above test support banner.

Remove extra "}". Should be 2 blank lines above test support banner.

5:36 pm

Sean Nash

What is d2?

What is d2?

5:36 pm

Sean Nash

should be "void sendOperationStatusEvent( void )"

should be "void sendOperationStatusEvent( void )"

5:36 pm

Sean Nash

Missing function header.

Missing function header.

5:34 pm

Sean Nash

Remove blank line.

Remove blank line.

5:31 pm

Sean Nash

Do we need to call this function on transition to consumables self-test so that op modes will get initial state? (Question applies throughout this code branch)

Do we need to call this function on transition to consumables self-test so that op modes will get initial state? (Question applies throughout this code branch)