add __eq__ method to SparseEmbedding

[anakin87]

Related Issues

While working on hybrid retrieval, I noticed we lack a proper way to compare SparseEmbedding objects.

Proposed Changes:

Add an __eq__ method to SparseEmbedding class to compare two SparseEmbedding objects.

How did you test it?

CI, new test

Checklist

• I have read the contributors guidelines and the code of conduct
• I have updated the related issue with new insights and changes
• I added unit tests and updated the docstrings
• I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
• I documented my code
• I ran pre-commit hooks and fixed any issue

[silvanocerza]

Would it make more sense to make it a dataclass at this point? 🤔

[anakin87]

@silvanocerza I don't expect many more changes and features to be added to this object.
Do you think it is better to have a dataclass?

[coveralls]

Pull Request Test Coverage Report for Build 8799970368

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.003%) to 90.109%

---

Totals
Change from base Build 8799941330:	0.003%
Covered Lines:	6286
Relevant Lines:	6976

---

💛 - Coveralls

[silvanocerza]

@anakin87 not essential probably but looking at the code the behaviour is really similar to a dataclass without being one.

[davidsbatista]

looks good to me 👍🏽