fix merge_attributions

[DanielSc4]

Description

The merge_attributions method is moved out of the FeatureAttributionOutput class as described in #209.

Related Issue

Fix #209

Type of Change

• 💥 Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• I've read the CODE_OF_CONDUCT.md document.
• I've read the CONTRIBUTING.md guide.
• I've successfully run the style checks using make fix-style.
• I've written tests for all new methods and classes that I created and successfully ran make test.
• I've written the docstring in Google format for all the methods and classes that I used.

[gsarti]

Note: this implements a breaking change for merging attributions, defining merge_attributions as a stand-alone function rather than a static method to prevent misleading usage from FeatureAttributionOutput instances.

[DanielSc4]

Oops, I wrote it without thinking. I just edited the description accordingly.

[DanielSc4]

I also report that the merge_attributions function could be moved to Main Functions in the documentation (here) since it is now callable with inseq.merge_attributions(...).

[gsarti]

@DanielSc4 I updated docs in inseq-team:fix-merge_attributions. You should be able to pull my commit in your fork, and it will be reflected here. I can proceed with the merge afterwards!

[DanielSc4]

The fork is now up-to-date w/ inseq-team/inseq:fix-merge_attributions! The PR should now be ready to be merged 🚀.

[gsarti]

LGTM!