Fisheye on leahi-dialin/tests/test_training.py

LEAHI-DIALIN-LDT-5152-1: closed by Zoltan Miskolci

Zoltan Miskolci — Tue, 11 Aug 2026 09:34:58 -0700

Zoltan Miskolci Closed LEAHI-DIALIN-LDT-5152-1

LEAHI-DIALIN-LDT-5152-1: comment by Zoltan Miskolci "I think that's what is usually shown, the first..."

Zoltan Miskolci — Mon, 10 Aug 2026 15:35:30 -0700

Zoltan Miskolci replied to Raghu Kallala in LEAHI-DIALIN-LDT-5152-1

I think that's what is usually shown, the first 7 char, but in this case it might have been a typo

LEAHI-DIALIN-LDT-5152-1: reviewers finished

Mon, 10 Aug 2026 15:07:00 -0700

Raghu Kallala finished reviewing LEAHI-DIALIN-LDT-5152-1

LEAHI-DIALIN-LDT-5152_[Dialin] Handler architecture refactor

LEAHI-DIALIN-LDT-5152-1: comment by Raghu Kallala "Was it a regulatory requirement before to have ..."

Raghu Kallala — Mon, 10 Aug 2026 14:29:02 -0700

Raghu Kallala commented on LEAHI-DIALIN-LDT-5152-1

Was it a regulatory requirement before to have 7 characters from commit ID or something we implemented?

LEAHI-DIALIN-LDT-5152-1: comment by Zoltan Miskolci "No need, in a different CR I have a module that..."

Zoltan Miskolci — Fri, 07 Aug 2026 15:48:20 -0700

Zoltan Miskolci replied to Michael Garthwaite in LEAHI-DIALIN-LDT-5152-1

No need, in a different CR I have a module that pulls the Alarm ID, Reject Reason and Msg Ids directly from the FW. After that's merged, this file won't be needed any more manual update.

For more information check LDT-5205 CR, file: utils\ enum_updater.py

LEAHI-DIALIN-LDT-5152-1: reviewers finished

Thu, 06 Aug 2026 22:21:37 -0700

Michael Garthwaite finished reviewing LEAHI-DIALIN-LDT-5152-1

LEAHI-DIALIN-LDT-5152_[Dialin] Handler architecture refactor

LEAHI-DIALIN-LDT-5152-1: comment by Michael Garthwaite "In FW we pre-populated Alarm IDs up to 300 to h..."

Michael Garthwaite — Thu, 06 Aug 2026 22:08:47 -0700

Michael Garthwaite commented on LEAHI-DIALIN-LDT-5152-1

In FW we pre-populated Alarm IDs up to 300 to help with the reservation process. Would this also benefit dialin if we reserve up to 300?

leahi-dialin (LDT-5205-dd_nvm_update): - Removed denali dialin test cases

Zoltan Miskolci — Wed, 29 Jul 2026 15:00:52 -0700

Zoltan Miskolci committed d10bf8f on branch LDT-5152-handler_refactor, LDT-5205-dd_nvm_update and 4 more in leahi-dialin

- Removed denali dialin test cases

· /leahi_dialin/common/dd_defs_proxy.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/common/td_defs.py

(-15, +15)

| History

| Source

| Diff

· /tests/DebugMsgCatcher.py

(-88, +0)

| History

| Source

| Diff

· /tests/air_trap_levels.py

(-44, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/dg_hd_switches_test.py

(-63, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/dg_heaters_test.py

(-81, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/dg_nvm_scripts.py

(-125, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/dg_tests.py

(-810, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/dg_valves_test.py

(-57, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/hd_air_bubbles_data.py

(-75, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/hd_blood_leak_data.py

(-137, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/hd_fluid_leak_state.py

(-39, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/hd_nvm_scripts.py

(-139, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/hd_test_pumps_open_loop.py

(-94, +0)

| History

| Source

| Diff

· /tests/depreciated_tests/hd_valves_test.py

(-160, +0)

| History

| Source

| Diff

… 51 more files in changeset.

LEAHI-DIALIN-LDT-5152-1: started by Dara Navaei

Dara Navaei — Mon, 20 Jul 2026 12:34:32 -0700

Dara Navaei started review LEAHI-DIALIN-LDT-5152-1

LEAHI-DIALIN-LDT-2428-1: closed by Sean Nash

Sean Nash — Tue, 21 Oct 2025 14:23:30 -0700

Sean Nash Closed LEAHI-DIALIN-LDT-2428-1

LEAHI-DIALIN-LDT-2428-1: reviewers finished

Tue, 21 Oct 2025 12:49:45 -0700

Michael Garthwaite and jpaguio (deleted user) finished reviewing LEAHI-DIALIN-LDT-2428-1

LEAHI-DIALIN-LDT-2428_Onboarding Test Message & Diag Screen View

LEAHI-DIALIN-LDT-2428-1: comment by Michael Garthwaite "after some thought, test_training.py will need ..."

Michael Garthwaite — Fri, 17 Oct 2025 10:38:28 -0700

Michael Garthwaite replied to Sean Nash in LEAHI-DIALIN-LDT-2428-1

after some thought, test_training.py will need to know where leahi dialin is if leahi_dialin is not installed in the venv. This is okay

LEAHI-DIALIN-LDT-2428-1: comment by Sean Nash "Do we not need to do this anymore?"

Sean Nash — Thu, 16 Oct 2025 10:24:49 -0700

Sean Nash replied to Michael Garthwaite in LEAHI-DIALIN-LDT-2428-1

Do we not need to do this anymore?

LEAHI-DIALIN-LDT-2428-1: comment by Michael Garthwaite "how come we need to add the path here?"

Michael Garthwaite — Thu, 16 Oct 2025 09:27:53 -0700

Michael Garthwaite commented on LEAHI-DIALIN-LDT-2428-1

how come we need to add the path here?

LEAHI-DIALIN-LDT-2428-1: reviewers left LEAHI-DIALIN-LDT-2428_Onboarding Test Message & Di...

Thu, 02 Oct 2025 15:06:41 -0700

Sean Nash left LEAHI-DIALIN-LDT-2428-1

LEAHI-DIALIN-LDT-2428_Onboarding Test Message & Diag Screen View

leahi-dialin (LDT-4734-temp-comp-conductivity): LDT-2428: Added training test script.

Sean Nash — Thu, 02 Oct 2025 08:29:32 -0700

Sean Nash committed 87891e5 on branch LDT-1195-dd-is-balancing-chamber-switching, LDT-1394-blood-flow-rate-overide-request and 103 more in leahi-dialin

LDT-2428: Added training test script.

· /tests/test_training.py

(-0, +41)

| History

| Source

LEAHI-DIALIN-LDT-2428-1: started by Dara Navaei

Dara Navaei — Wed, 01 Oct 2025 12:01:26 -0700

Dara Navaei started review LEAHI-DIALIN-LDT-2428-1