feat: Stronger Types

[besaleli]

No description provided.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 68.32061% with 83 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
encoderfile-core/src/cli.rs	0.00%	20 Missing ⚠️
encoderfile-core/src/server.rs	0.00%	16 Missing ⚠️
encoderfile-core/src/common/embedding.rs	0.00%	6 Missing ⚠️
encoderfile-core/src/common/sentence_embedding.rs	0.00%	6 Missing ⚠️
...derfile-core/src/common/sequence_classification.rs	0.00%	6 Missing ⚠️
...ncoderfile-core/src/common/token_classification.rs	0.00%	6 Missing ⚠️
encoderfile-core/src/factory.rs	0.00%	6 Missing ⚠️
encoderfile-core/src/runtime/state.rs	78.57%	6 Missing ⚠️
encoderfile-core/src/transport/grpc/mod.rs	25.00%	6 Missing ⚠️
encoderfile/src/model.rs	57.14%	3 Missing ⚠️
... and 2 more

Files with missing lines	Coverage Δ
encoderfile-core/src/common/model_type.rs	100.00% <100.00%> (ø)
encoderfile-core/src/dev_utils/mod.rs	100.00% <100.00%> (ø)
encoderfile-core/src/inference/embedding.rs	100.00% <100.00%> (ø)
...coderfile-core/src/inference/sentence_embedding.rs	100.00% <100.00%> (ø)
...file-core/src/inference/sequence_classification.rs	100.00% <100.00%> (ø)
...derfile-core/src/inference/token_classification.rs	96.61% <100.00%> (+0.05%)	⬆️
encoderfile-core/src/services/embedding.rs	100.00% <100.00%> (ø)
encoderfile-core/src/services/model_metadata.rs	100.00% <100.00%> (ø)
...ncoderfile-core/src/services/sentence_embedding.rs	100.00% <100.00%> (ø)
...ncoderfile-core/src/transforms/engine/embedding.rs	90.90% <100.00%> (+0.11%)	⬆️
... and 20 more

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[codspeed-hq]

CodSpeed Performance Report

Merging #172 will not alter performance

_{Comparing feat/stronger-types-backup (c1c57b3) with main (fd5e8f7)¹}

Summary

✅ 20 untouched
⏩ 20 skipped²

Footnotes

1. No successful run was found on main (5590429) during the generation of this report, so fd5e8f7 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

2. 20 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[javiermtorres]

LGTM!

[javiermtorres]

Does it make sense to make a general state() delegating to some fn in the trait?