ActivityWatch / aw-core

Core library for ActivityWatch
Mozilla Public License 2.0
48 stars 46 forks source link

feat: made all queries range inclusive #102

Closed ErikBjare closed 3 years ago

ErikBjare commented 3 years ago

Should fix https://github.com/ActivityWatch/activitywatch/issues/609

codecov[bot] commented 3 years ago

Codecov Report

Merging #102 (a82b745) into master (f04a450) will increase coverage by 0.02%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
+ Coverage   90.75%   90.78%   +0.02%     
==========================================
  Files          34       34              
  Lines        1655     1660       +5     
  Branches      254      257       +3     
==========================================
+ Hits         1502     1507       +5     
  Misses        118      118              
  Partials       35       35              
Impacted Files Coverage Δ
aw_datastore/datastore.py 100.00% <100.00%> (ø)
aw_datastore/storages/memory.py 93.44% <100.00%> (-0.77%) :arrow_down:
aw_datastore/storages/peewee.py 99.29% <100.00%> (+0.07%) :arrow_up:
aw_datastore/storages/sqlite.py 96.62% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f04a450...a82b745. Read the comment docs.

ErikBjare commented 3 years ago

Tested on real data now. Works, but it's horribly slow. Trying to figure out indexes and other optimizations...

ErikBjare commented 3 years ago

Performance isn't great, tried improving it in a lot of different ways, but didn't make any spectacular progress.

At least it works correctly now. Merging.