search

evidentlyai / evidently

Evidently is ​​an open-source ML and LLM observability framework. Evaluate, test, and monitor any AI-powered system or data pipeline. From tabular data to Gen AI. 100+ metrics.
https://www.evidentlyai.com/evidently-oss
Apache License 2.0
5.2k stars 586 forks source link

Assorted refactorings of the collector and UI app #1045

Closed provinzkraut closed 5 months ago

provinzkraut commented 6 months ago

Hi all, this is a bunch of updates pertaining various things introduced in #995, when moving from FastAPI to Litestar.

Some of them are simple refactorings to simplify code and using Litestar best-practices, others should provide a real world performance boost.

I'll try to add comments to the various changes to explain why they have been made.

There's still some things that could be improved, but given my lack of familiarity with the codebase, I didn't want to go ahead and change too many things in code that's (as far as I can tell?) not (yet?) covered by unit tests.

provinzkraut commented 6 months ago

@mike0sv I've reverted the changes as discussed

DimaAmega commented 5 months ago

@provinzkraut Let's merge this pr