Coverage

[gumityolcu]

Hello,

Coverage is at 90 %

With the following caveats:

1- We omit files that are not used anywhere currently:

• src/utils/explanations.py

• src/utils/transforms.py

• src/utils/datasets/transformed/sample.py

• src/utils/cache.py

• src/utils/datasets/activation_dataset.py

• src/utils/datasets/indexed_subset.py

• src/explainers/functional.py

2- We don't yet test:

• Errors and warnings (this means all of utils/validation.py is also not tested)
• reset functions, state_dict and load_state_dict
• Metric and GlobalMetric base classes, because they just initialize
• functions in utils/functions. Kendall and Spearman correlations and dot and cos similarity functions. I felt they are a little too obviously correctly implemented
• functions in common.py because i felt they are not urgent

[gumityolcu]

Hello Dilya, thanks for the feedback!

incorporated most of it.

The only thing is, I don't get what you mean by "tmp_path" because currently in main it is an extra parameter that does not get a value. So i was confused and I removed it. You can either tell me or do your own edit about it (if it's a quick fix as i imagine)

Any other feedback is of course also welcome!

[gumityolcu]

Hi

I tried to name everything using class and method names.

I added in the tmp_path fixture 😅