leahi-dialin

Clone Tools

Stats

Commits this week: 21
Total Committers: 16
Last Commit: 14 Jul
Commits this week: 21

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 29 Jun
12:24 pm

Zoltan Miskolci

Merge branch 'LDT-4539-event_on_service_change' into staging

Options
    • -2
    • +12
    /leahi_dialin/common/dd_defs.py
12:24 pm

Zoltan Miskolci

Merge branch 'LDT-4160-alarm_table' into staging

Options
    • -0
    • +2
    /leahi_dialin/common/msg_ids.py
    • -2
    • +2
    /leahi_dialin/utils/abstract_classes.py
11:29 am

Zoltan Miskolci

Merge branch 'LDT-4539-event_on_service_change' into develop

Options
    • -2
    • +12
    /leahi_dialin/common/dd_defs.py
11:26 am

Zoltan Miskolci

Merge branch 'LDT-4160-alarm_table' into develop

Options
    • -0
    • +2
    /leahi_dialin/common/msg_ids.py
    • -42
    • +109
    /leahi_dialin/td/modules/alarms.py
    • -2
    • +2
    /leahi_dialin/utils/abstract_classes.py
11:23 am

Zoltan Miskolci

Merge branch 'LDT-4878-buildsh' into develop

Options
11:20 am

Zoltan Miskolci

Updated build.sh

Options
    • -6
    • +13
    /build.sh
11:01 am

Dara Navaei

LEAHI-DIALIN-LDT-4878_(Dialin) The wheel file generated by the build.sh command failed to install
LEAHI-DIALIN-LDT-4878_(Dialin) The wheel file generated by the build.sh command failed to install
10:56 am

Tiffany Mejia and Michael Garthwaite

10:50 am

Michael Garthwaite

10:44 am

Zoltan Miskolci

fixed indexing for eventlist enum

Options
    • -6
    • +6
    /leahi_dialin/common/dd_defs.py
10:42 am

Zoltan Miskolci

docu header

Options
    • -0
    • +3
    /leahi_dialin/td/modules/alarms.py
9:44 am

Zoltan Miskolci

Being generic is the whole point of it. It was renamed because this is a dialin dedicated to Leahi unit and calling the messages DenaliMessage is misleading. It was renamed to CanMessage for 2 reas...

Being generic is the whole point of it.
It was renamed because this is a dialin dedicated to Leahi unit and calling the messages DenaliMessage is misleading.
It was renamed to CanMessage for 2 reason:
1. This is a Leahi only dialin, so there is no need to differentiate between different type of Can Messages as there would be only 1 type
2. Calling it CanMessage will make it more future compatible when the dialin get copied over for a new unit development (unlikely in the foreseeable future but possible), removing the need to rename it again

9:43 am

Zoltan Miskolci

There is no Leahi or Denali CANChannels. There are CAN channels and the naming is what that channel is being used for. The collision occurs in the FW and the FW handles it before sending out a mess...

There is no Leahi or Denali CANChannels.
There are CAN channels and the naming is what that channel is being used for.
The collision occurs in the FW and the FW handles it before sending out a message.
When reading a CAN, there are no collisions, so there are no priorities. So I have no clue what you want here.

9:39 am

Zoltan Miskolci

done

done

9:39 am

Zoltan Miskolci

I have no idea what you are referring to as the link is not working. The need to define the type and the name of the data being read is necessary, so if it's not here it will be somewhere else, tha...

I have no idea what you are referring to as the link is not working.
The need to define the type and the name of the data being read is necessary, so if it's not here it will be somewhere else, that we can't escape.
Exporting that to different files won't serve a purpose and will only increase complexity as when debugging you will need to reference a separate file.
For the defs it made sense as they are used multiple places, but what a message contains is a single use place, so storing it separately is meaningless as it won't have any use outside of this single function.

9:18 am

Zoltan Miskolci

Merge branch 'staging' into LDT-4484-dry_self_test

Options
    • -10
    • +14
    /leahi_dialin/td/treatment_delivery.py
Friday 26 Jun
4:56 pm

Raghu Kallala

4:26 pm

Zoltan Miskolci

removed some unnecessary entry from calibration records

Options
    • -0
    • +18
    /leahi_dialin/dd/modules/records.py
4:21 pm

Zoltan Miskolci

fixed

fixed

4:21 pm

Zoltan Miskolci

fixed typo

Options
    • -4
    • +4
    /leahi_dialin/td/modules/battery.py
4:15 pm

Vendor - TEL - Arpita Srivastava

updates

Options
    • -23
    • +39
    /leahi_dialin/dd/modules/records.py
3:17 pm

Dara Navaei

LEAHI-DIALIN-LDT-4410_Dialysate Composition (Mixing) - FW Implementation - 3/1:
LEAHI-DIALIN-LDT-4410_Dialysate Composition (Mixing) - FW Implementation - 3/1:
3:11 pm

Vendor - TEL - Jashwant Gantyada

LDT-4410 updated broadcast data for bal chamber

Options
    • -12
    • +10
    /leahi_dialin/dd/modules/balancing_chamber.py
2:13 pm

Raghu Kallala

fix typo

fix typo

1:42 pm

Zoltan Miskolci

Updated Alarm Priority data type

Options
    • -1
    • +2
    /leahi_dialin/td/modules/alarms.py
10:18 am

Dara Navaei

LEAHI-DIALIN-LDT-4872_[Request] Dialin module for Patient Vitals
LEAHI-DIALIN-LDT-4872_[Request] Dialin module for Patient Vitals
Thursday 25 Jun
5:33 pm

Behrouz NematiPour

I am happy to see that this old code is improving. I would recommend using the Conversion module instead of strings and DataTypes, as it handles everything automatically. Examples can be found in t...

I am happy to see that this old code is improving.
I would recommend using the Conversion module instead of strings and DataTypes, as it handles everything automatically.
Examples can be found in the following file in this CR:
https://devapps.diality.us/cru/#LEAHI-DIALIN-LDT-4484-1CFR-84688

5:19 pm

Behrouz NematiPour

Please add a docstring for the function. including the intention and parameters with their intentions.

Please add a docstring for the function.
including the intention and parameters with their intentions.

5:17 pm

Behrouz NematiPour

Same for CanChannels. The Channels we are referring to here are Leahi's or Denali's. Channels mainly have no meaning beyond their priorities when a collision occurs.

Same for CanChannels.
The Channels we are referring to here are Leahi's or Denali's.
Channels mainly have no meaning beyond their priorities when a collision occurs.

5:15 pm

Behrouz NematiPour

I recommend DenaliMessage or LeahiMessage, as CanMessage is generic, whereas Leahi/Denali indicate that the protocol is also implemented in the message we are sending. I'm not sure why the name was...

I recommend DenaliMessage or LeahiMessage, as CanMessage is generic, whereas Leahi/Denali indicate that the protocol is also implemented in the message we are sending.
I'm not sure why the name was changed, as it is a lot of change.
Also, there is no CAN Message.
There is a CAN Frame and a Leahi/Denali Message.