Skip to content

Fix/explanation methods#85

Open
liz-is wants to merge 3 commits into
devfrom
fix/explanation_methods
Open

Fix/explanation methods#85
liz-is wants to merge 3 commits into
devfrom
fix/explanation_methods

Conversation

@liz-is
Copy link
Copy Markdown
Contributor

@liz-is liz-is commented Mar 13, 2025

Resolves #73 - however sadly this only fixes two Pyright complaints that were associated with using _Explanation__global.

Pyright still complains when _spatial is used, and I don't understand the relevance of the error it reports:

 "None" is not iterable
  "__iter__" method not defined

(I would hold off on merging this until after #48 is merged, it's not urgent)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 13, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py00100% 
rex_wrapper.py16160%3–5, 7–9, 12, 14–16, 18, 20, 22–24, 26
explanation
   evaluation.py77988%27, 52–56, 58–59, 118
   explanation.py1703778%27, 34, 43, 73, 93, 96, 187, 191–192, 195–196, 199–201, 203–207, 209, 213, 226, 253, 261, 269, 277, 287–288, 295, 305–307, 314, 317–318, 321, 323
   multi_explanation.py1736264%43, 57, 59, 86–88, 90–92, 94–97, 99–102, 105, 107–108, 115, 117–118, 120, 148, 192–199, 201, 205, 211–213, 216, 219, 221–226, 229–236, 252, 266–268, 274, 277–279
   rex.py1956467%60–61, 75, 89, 118, 154–155, 186, 199, 279–280, 344, 347, 353–354, 360–363, 369, 379–382, 384–385, 388–389, 391–392, 395–397, 399–400, 403–405, 412–413, 450, 477, 479–480, 483, 486–501, 509–510, 514
input
   config.py31911863%97, 150, 168, 170, 174–175, 179, 208, 213, 217, 223, 227, 233, 239, 247, 254, 260, 267, 269, 275, 282, 289, 295, 301, 306, 312, 319, 325, 327, 331–333, 339, 343, 345, 347, 361, 363, 375–386, 388–394, 396, 400, 402–404, 406–408, 410, 412, 425, 535–537, 543–548, 550, 552, 554–555, 557–558, 560–562, 564–566, 570–571, 573–574, 576–579, 582–587, 590, 592–594, 599, 601–603, 605, 607, 609, 611–612, 614, 700
   input_data.py1754077%52–53, 62, 65, 68, 71, 81, 95–96, 98, 102, 110, 131–132, 134–135, 138–143, 147–152, 164, 168, 185, 202, 208–209, 241, 243, 248–249, 251–252
   onnx.py973959%41, 61–62, 64–65, 83–85, 98, 100–102, 105, 108–110, 112–113, 122, 124, 126, 135–138, 151–152, 177–178, 180–184, 187, 189–191, 194
mutants
   box.py1712386%67–71, 116, 168, 178, 253, 266, 280, 349–350, 361, 372, 392, 449–450, 465, 481, 491–492, 518
   distributions.py541964%28–31, 36–38, 40, 45–46, 49, 55, 61, 75, 81–82, 87–89
   mutant.py912968%9–10, 52–53, 79, 90, 99, 119, 122–125, 127–128, 131, 133–137, 139, 141–146, 148–149
   occlusions.py25388%61–63
output
   database.py151497%73, 172–173, 198
   visualisation.py37117153%27–29, 31–45, 47–55, 57–63, 65–66, 68–69, 71, 86, 117, 122, 128, 130–135, 137–140, 142, 146–149, 151–157, 159, 161–163, 167–175, 177–178, 180–182, 186, 188, 190–191, 193, 195, 205–206, 208, 263–264, 266, 269–270, 276, 281, 287, 289–291, 293–295, 297–298, 300–302, 304, 306, 315, 317, 324, 326, 329–331, 348–350, 355, 360–361, 444, 446, 448, 451, 453, 455, 458, 460, 462, 464, 467, 482–484, 486, 524, 539, 545, 556–557, 564–565, 584, 593, 603, 606, 614–617, 620, 623, 634, 657, 663–673
responsibility
   prediction.py38586%29, 36, 38, 41, 44
   resp_maps.py881681%9–10, 33–34, 38–39, 48, 51, 71, 97, 107, 118, 131–132, 138–139
   responsibility.py1373375%15–16, 45, 76, 78–90, 96, 98, 184, 234–236, 240, 270–271, 274–275, 278, 290–291, 293–294
utils
   _utils.py1213769%26, 46, 51–52, 55, 60–61, 64, 73, 78–79, 82, 86–87, 91–92, 94, 101, 111, 113, 115, 141–143, 149–151, 163, 165, 167, 169, 172, 177–178, 180, 188, 192
   logger.py14750%11–16, 18
TOTAL248373270% 

@liz-is liz-is requested a review from kellino March 25, 2025 15:47
@liz-is liz-is force-pushed the fix/explanation_methods branch from 29c3db6 to 9fb5764 Compare March 31, 2025 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Swap double underscore to single underscore in Explanation methods

1 participant