Closed simonhkswan closed 5 months ago
Attention: 7 lines
in your changes are missing coverage. Please review.
Comparison is base (
68fbadb
) 79.9% compared to head (edb0d2e
) 79.0%.:exclamation: Current head edb0d2e differs from pull request most recent head 0be0676. Consider uploading reports for the commit 0be0676 to get more accurate results
Kudos, no new issues were introduced!
0 New issues
0 Security Hotspots
75.3% Coverage on New Code
0.0% Duplication on New Code
Inserting data into the database was quite slow, especially when we were commiting each individual value into the database in a diff-correlation metric. For every value we were also querying the database for a dataset_id and a metric_id.
So to speed up the process we now use a single session.commit() for each of the DataFrame metrics. We also use cachetools on the database/metric id functions. It's much faster locally and likely and even better speedup when the database is remote.