365 potentially remove interactionvaluesvalues

[Advueu963]

This pull request addresses #365 and #385 by changing InteractionValues.values and InteractionValues.interaction_lookup into properties. Introducing InteractionValues.interactions as the main part working with interactions.
We remove finalize_computed_interactions and incorporate it into the __init__ of InteractionValues.

Adding Logic into shapiq.Game:

• Introduced the .coalition_values dictionary, mapping coalitions to their game values
• Updated Loading and Saving to be based on .coalition_values.
• Updated .value_storage and .coalition_lookup to be properties of the Game class

Removal of finalize_computed_interactions:

• Updated all approximate methods in various approximator modules (owen.py, stratified.py, montecarlo/_base.py, permutation/sii.py, permutation/stii.py, permutation/sv.py, regression/_base.py, sparse/_base.py) to directly return InteractionValues with the target_index property included. This eliminates the need for the finalize_computed_interactions function. [1] [2] [3] [4] [5] [6] [7] [8]

• Updated the explain_function in tabular.py and tree/explainer.py to replace calls to finalize_computed_interactions with direct instantiation of InteractionValues. [1] [2]

Simplification of imports:

• Removed finalize_computed_interactions from imports in all affected files, as it is no longer used. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

These changes improve the maintainability of the code by reducing complexity and removing unnecessary dependencies.

[mmschlk]

This is amazing ... but we really gotta be careful integrating this haha 😂

[mmschlk]

We could add this logic actually also to our game class ... I don't see if there is something which speaks against this.

[Advueu963]

The coda quality error seems to be originate in tests/shapiq_games/test_tabular_local_xai.py, yet it pre-commit does not raise an issue locally, nor can I induce this change as pre-commit then changes it back

[mmschlk]

The coda quality error seems to be originate in tests/shapiq_games/test_tabular_local_xai.py, yet it pre-commit does not raise an issue locally, nor can I induce this change as pre-commit then changes it back

I also do not get any local pre-commit changes. I will try to print the changes in the workflow and see what happens.

[mmschlk]

I hade one comment against defensive type checks. Otherwise this is good to go! 🫶🫡