leahi-dialin / leahi_dialin

utils

Clone Tools

Stats

Commits this week: 2
Total Committers: 4
Last Commit: 16:08
Commits this week: 2

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
Today
4 hrs ago

Zoltan Miskolci

4 hrs ago

Zoltan Miskolci

Merge branch 'LDT-3904-opmode-the-operation-mode-broad' into staging

Options
    • -0
    • +157
    ./abstract_classes.py
  2. … 12 more files in changeset.
10 hrs ago

Zoltan Miskolci

Merge branch 'LDT-3904-opmode-the-operation-mode-broad' into develop

Options
    • -4
    • +1
    ./__init__.py
    • -0
    • +157
    ./abstract_classes.py
  3. … 67 more files in changeset.
12 hrs ago

Raghu Kallala

Friday 29 May
10:05 am

Michael Garthwaite

Thursday 21 May
5:09 pm

Zoltan Miskolci

references to Denali has been removed

references to Denali has been removed

7:44 am

Raghu Kallala

Shouldn't we use Leahi instead of Denali?

Shouldn't we use Leahi instead of Denali?

Friday 08 May
8:47 am

Zoltan Miskolci

- Added enums to utils init to cover most import scenarios in other projects

Options
    • -0
    • +1
    ./__init__.py
Wednesday 06 May
4:07 pm

Zoltan Miskolci

- Updated Event enum - Updated Datatypes enum - Added some debugging print to abstract class - Removed some init import to remove cyclic dependencies - Updated alarms

Options
    • -28
    • +0
    ./__init__.py
    • -10
    • +39
    ./abstract_classes.py
    • -23
    • +2
    ./enums/__init__.py
  4. … 11 more files in changeset.
Tuesday 05 May
2:25 pm

Zoltan Miskolci

- Updated DD and TD module - Updaterd TDValvePosition enum

Options
    • -4
    • +4
    ./abstract_classes.py
  2. … 54 more files in changeset.
Monday 04 May
5:46 pm

Dara Navaei

LEAHI-DIALIN-LDT-3904_[OpMode] The operation mode broadcasts does not return the 3rd and 4th level modes
LEAHI-DIALIN-LDT-3904_[OpMode] The operation mode broadcasts does not return the 3rd and 4th level modes
5:24 pm

Zoltan Miskolci

- Handler and import refactoring of DD module

Options
    • -0
    • +128
    ./abstract_classes.py
    • -97
    • +1
    ./base.py
    • -0
    • +23
    ./enums/__init__.py
    • -0
    • +29
    ./enums/alarm_enum.py
    • -0
    • +51
    ./enums/dialin_enum.py
    • -1
    • +2
    ./nv_ops_utils.py
  7. … 40 more files in changeset.
Thursday 12 Mar
2:03 pm

Michael Garthwaite

Merge branch 'staging' into LDT-2655-fix_conductivity_override

Options
    • -1
    • +1
    ./__init__.py
  2. … 6 more files in changeset.
Monday 09 Feb
10:34 am

Zoltan Miskolci

Saturday 07 Feb
4:46 pm

Dara Navaei

Friday 06 Feb
9:25 am

Zoltan Miskolci

We agreed to change back the enums into the static variation. Why would that cover a class that has been implemented to mask the enums from direct access, so if they need to be changed to dynamic o...

We agreed to change back the enums into the static variation.
Why would that cover a class that has been implemented to mask the enums from direct access, so if they need to be changed to dynamic or renamed or anything, the other repositories that are using them through this class won't be affected or need any update.

Thursday 05 Feb
3:38 pm

Dara Navaei

Did we not decide to revert all the changes back and the redesign the whole concept again?

Did we not decide to revert all the changes back and the redesign the whole concept again?

3:35 pm

Behrouz NematiPour

Wednesday 04 Feb
3:05 pm

Zoltan Miskolci

- Removed ignore and _str_list from base class, added empty _str_list to all childs to not throw an error when from_str is called on them

Options
    • -6
    • +4
    ./base.py
    • -1
    • +2
    ./nv_ops_utils.py
  3. … 6 more files in changeset.
1:32 pm

Zoltan Miskolci

Removed ignore from dialinenum

Options
    • -1
    • +1
    ./base.py
Tuesday 03 Feb
3:55 pm

Behrouz NematiPour

Look in the td_defs.py for the same comment.

Look in the td_defs.py for the same comment.

3:55 pm

Behrouz NematiPour

Look in the td_defs.py for the same comment.

Look in the td_defs.py for the same comment.

3:55 pm

Behrouz NematiPour

Look in the td_defs.py for the same comment.

Look in the td_defs.py for the same comment.

3:55 pm

Behrouz NematiPour

Look in the td_defs.py for the same comment.

Look in the td_defs.py for the same comment.

3:54 pm

Behrouz NematiPour

This design is incorrect, For common codes, a proxy or a wrapper class should have been implemented. We will keep it for now to unblock the rest of the team.

This design is incorrect,
For common codes, a proxy or a wrapper class should have been implemented.
We will keep it for now to unblock the rest of the team.

3:54 pm

Behrouz NematiPour

This is incorrect. We should not create many objects for all the Enum classes in an API, and we should not use memory that may not even be in use. We will keep it for now to unblock the rest of the...

This is incorrect.
We should not create many objects for all the Enum classes in an API, and we should not use memory that may not even be in use.
We will keep it for now to unblock the rest of the team.

2:27 pm

Zoltan Miskolci

When building locally without all the parameters the generated name throws an error during python pip install. So this ensures that if no parameter is added, it uses the default name which is compa...

When building locally without all the parameters the generated name throws an error during python pip install.
So this ensures that if no parameter is added, it uses the default name which is compatible with pip install.
The " -+. " sequence is throwing an error.

2:25 pm

Zoltan Miskolci

removed commented out line

removed commented out line

2:20 pm

Zoltan Miskolci

It's a class containing all the enums of the def file. Yes we need them as many other Framework and huge amount of code are using them.

It's a class containing all the enums of the def file.
Yes we need them as many other Framework and huge amount of code are using them.

2:20 pm

Zoltan Miskolci

Obviously not. You were reminding me about backwards compatibility. This variable and class ensures that even with this major refactor to the enum structure, the current codebase remains working.

Obviously not.
You were reminding me about backwards compatibility. This variable and class ensures that even with this major refactor to the enum structure, the current codebase remains working.