leahi-dialin

Clone Tools

Stats

Commits this week: 18
Total Committers: 16
Last Commit: 18 Jul
Commits this week: 18

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 06 Jul
11:34 am

Zoltan Miskolci

Merge branch 'staging' into LDT-4872-patient_vitals

Options
11:34 am

Zoltan Miskolci

Added new enums

Options
    • -0
    • +45
    /leahi_dialin/common/generic_defs.py
    • -0
    • +13
    /leahi_dialin/common/td_defs.py
    • -0
    • +4
    /leahi_dialin/common/td_defs_proxy.py
Thursday 02 Jul
3:39 pm

Nicholas Ramirez

3:38 pm

Behrouz NematiPour and Stephen Quong

3:36 pm

Nicholas Ramirez

bn approves

bn approves

3:35 pm

Nicholas Ramirez

3:31 pm

Stephen Quong

3:14 pm

Vinayakam Mani

3:11 pm

Vinayakam Mani

2:13 pm

Michael Garthwaite

2:10 pm

Zoltan Miskolci

Merge branch 'LDT-4873' into develop

Options
    • -2
    • +3
    /leahi_dialin/common/test_config_defs.py
2:10 pm

Zoltan Miskolci

Added test config for conductivity alarm

Options
    • -2
    • +3
    /leahi_dialin/common/test_config_defs.py
2:02 pm

Dara Navaei

LEAHI-DIALIN-LDT-4873_(Request) Add config to disable conductive level sensor fault
LEAHI-DIALIN-LDT-4873_(Request) Add config to disable conductive level sensor fault
1:19 pm

Vendor - TEL - Varshini Nagabooshanam

9:54 am

Zoltan Miskolci

Wednesday 01 Jul
8:58 pm

Vendor - TEL - Sameer Poyil , Vinayakam Mani and Zoltan Miskolci

8:58 pm

Vendor - TEL - Sameer Poyil

6:26 pm

Vendor - TEL - Sameer Poyil

LDT-4614 dialin support for mixing-algo

Options
    • -0
    • +13
    /leahi_dialin/dd/modules/mixing_cntrl.py
3:25 pm

Zoltan Miskolci

added reject reason handlers

Options
    • -6
    • +282
    /leahi_dialin/dd/modules/records.py
3:17 pm

Dara Navaei

LEAHI-DIALIN-LDT-4614_Dialysate Composition (Dry Bicarbonate) - FW Implementation - 5/1:
LEAHI-DIALIN-LDT-4614_Dialysate Composition (Dry Bicarbonate) - FW Implementation - 5/1:
3:02 pm

Zoltan Miskolci

The approach of what this code does and the conversion module are similar in function but a differ on maintenance and flexibility. 1. The conversion module's bytearray to value methods work, but th...

The approach of what this code does and the conversion module are similar in function but a differ on maintenance and flexibility.
1. The conversion module's bytearray to value methods work, but they require manual logistic around the indexing. It's the user's responsibility to handle where the reading starts and you need to code the handler for that (previous approach with MsgFieldPositions), It's inflating the method. I worked hard to remove the need for indexing where the value starts. Instead of the previous struct.unpack approach, where you needed to maintain the type, you will be needing to maintain the conversion methods that are being called, so it would be the same as before, you still need to mind the indexing, which was the main reason of changing from this approach.

2. In the current approach, you declare the type and pass it to a processor that dynamically handling the unpack type (based on the set datatype) and the indexing for that. This works much better and easier to maintain, as you can just insert a new value when it's added to the FW, and don't need to adjust all your indexes for the reading.

Example #1, changing build type from U16 to U08:

  • with the conversion approach, you need to change the function used, and update the starting index for all the following methods.
  • with the current approach you change the Datatype.U16 to Datatype.U08 and that's all you need to do.


Example #2: adding a new variable fpga_micro after fpga_minor, with U08 type:

  • with the conversion approach, you add the bytearray_to_byte, update the indexes for all the following reads.
  • with the current approach you insert an element after the fpga_minor with a value ('fpga_micro', Datatype.U08), and nothing more needed
3:00 pm

Michael Garthwaite

fixed. thanks!

fixed. thanks!

3:00 pm

Michael Garthwaite

fixed. thanks

fixed. thanks

2:58 pm

Michael Garthwaite

LDT-4734: code review comments

Options
    • -0
    • +5
    /leahi_dialin/dd/modules/conductivity_sensors.py
    • -0
    • +2
    /leahi_dialin/fp/modules/conductivity_sensors.py
2:56 pm

Michael Garthwaite

Merge branch 'staging' into LDT-4734-temp-comp-conductivity

Options
2:40 pm

Zoltan Miskolci

Add uncompensated conductivity to the init too

Add uncompensated conductivity to the init too

2:40 pm

Zoltan Miskolci

Add uncompensated conductivity to the init too

Add uncompensated conductivity to the init too

2:25 pm

Zoltan Miskolci

2:08 pm

Zoltan Miskolci

Merge branch 'LDT-4505' into staging

Options
1:11 pm

Michael Garthwaite

LDT-4734: updates to conductivity message

Options
    • -3
    • +5
    /leahi_dialin/common/dd_defs.py
    • -0
    • +5
    /leahi_dialin/dd/modules/conductivity_sensors.py
    • -14
    • +8
    /leahi_dialin/fp/modules/conductivity_sensors.py