Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable analyzing nested input- and output-dicts #212

Merged
merged 16 commits into from
Feb 5, 2023
Merged

Enable analyzing nested input- and output-dicts #212

merged 16 commits into from
Feb 5, 2023

Commits on Jan 13, 2023

  1. enable analyzing nested input- and output dicts

    @snimu
    snimu committed Jan 13, 2023
    Configuration menu
    Copy the full SHA
    4f540b8 View commit details
    Browse the repository at this point in the history

  2. enable analyzing nested input- and output dicts

    @snimu
    snimu committed Jan 13, 2023
    Configuration menu
    Copy the full SHA
    ef0e4ed View commit details
    Browse the repository at this point in the history

  3. skip tests that require torch v1.8 or above when an older version is … 

    …installed
    @snimu
    snimu committed Jan 13, 2023
    Configuration menu
    Copy the full SHA
    4d17355 View commit details
    Browse the repository at this point in the history

  4. add test for highly nested dicts, fix error found by it 

    - `elem_bytes` in `LayerInfo.calculate_size(...)` didn't work for nested dicts
    @snimu
    snimu committed Jan 13, 2023
    Configuration menu
    Copy the full SHA
    38e2af0 View commit details
    Browse the repository at this point in the history

Commits on Jan 16, 2023

  1. LayerInfo.calculate_size.extract_tensor now works with dict properly 

    - adapted highly_nested_dict_model.out accordingly
    @snimu
    snimu committed Jan 16, 2023
    Configuration menu
    Copy the full SHA
    a96e995 View commit details
    Browse the repository at this point in the history

  2. simplified test_highly_nested_dict_model

    @snimu
    snimu committed Jan 16, 2023
    Configuration menu
    Copy the full SHA
    6931e39 View commit details
    Browse the repository at this point in the history

Commits on Jan 17, 2023

  1. LayerInfo.calculate_size.extract_tensor now works properly for obje… 

    …cts with `tensor`-attribute

- Found error in new testcase that comes with this commit
    @snimu
    snimu committed Jan 17, 2023
    Configuration menu
    Copy the full SHA
    ad7ee14 View commit details
    Browse the repository at this point in the history

  2. Add docstring to test to explain what exactly it tests

    @snimu
    snimu committed Jan 17, 2023
    Configuration menu
    Copy the full SHA
    81167cb View commit details
    Browse the repository at this point in the history

  3. test all edge-cases of LayerInfo.calculate_size.extract_tensor

    @snimu
    snimu committed Jan 17, 2023
    Configuration menu
    Copy the full SHA
    4d31c90 View commit details
    Browse the repository at this point in the history

  4. use dim=0 in F.softmax explicitely (implicit use depreciated)

    @snimu
    snimu committed Jan 17, 2023
    Configuration menu
    Copy the full SHA
    b51d029 View commit details
    Browse the repository at this point in the history

  5. replace custom torchversion_at_least with packaging.version.parse

    @snimu
    snimu committed Jan 17, 2023
    Configuration menu
    Copy the full SHA
    80bb192 View commit details
    Browse the repository at this point in the history

  6. modify EdgecaseInputOutputModel to increase test-coverage 

    missing:

- not hasattr(inputs, "__getitem__")
- last return
    @snimu
    snimu committed Jan 17, 2023
    Configuration menu
    Copy the full SHA
    55aee3e View commit details
    Browse the repository at this point in the history

Commits on Jan 21, 2023

  1. use torch_nested-package to simplify LayerInfo.calculate_size 

    - torch_nested has 99.something% test-coverage
- Makes test-coverage for this package much easier
- Increases readability & extensibility
    @snimu
    snimu committed Jan 21, 2023
    Configuration menu
    Copy the full SHA
    6daa9f1 View commit details
    Browse the repository at this point in the history

  2. Move back from using torch-nested. Fix and use nested_list_size ins… 

    …tead

- Fixes issue#141
- Increases test-coverage
- Produces more plausible output for some cases
    @snimu
    snimu committed Jan 21, 2023
    Configuration menu
    Copy the full SHA
    563b8ba View commit details
    Browse the repository at this point in the history

Commits on Feb 5, 2023

  1. Fix problem with accessing of dicts 

    Fix [issue#214](#215)
    @snimu
    snimu committed Feb 5, 2023
    Configuration menu
    Copy the full SHA
    a0a7dbc View commit details
    Browse the repository at this point in the history

  2. Install compressai in workflows

    @snimu
    snimu committed Feb 5, 2023
    Configuration menu
    Copy the full SHA
    116202f View commit details
    Browse the repository at this point in the history