get_events(b, limit = None) returns only 100 events. about aw-client

[johan-bjareholt]

If i remember correctly our reason for setting the limit to 100 was so activitywatch wouldn't take up all CPU and disk usage by dumping the whole bucket if someone had run activitywatch for a long period of time. Now when i think of it it's kind of stupid, but still valid I guess. At least we should document it somewhere.

@ErikBjare Should we remove the default 100 event limit or keep it?

Instead of setting limit = 10**6 you can use limit = -1 to fetch all events.

By the way @nikanar, if you havn't seen it yet you can have a nice view of the REST api at localhost:5600/api, could be useful.

[ErikBjare]

The default limit is 100 by default for very reason that Johan explained.

That being said, it might be better to have 100 as the default argument in ´aw-client´ and make ´limit=None´ mean that ´limit=-1´ is passed to the API.

And again @nikanar, I can't thank you enough for the work you are doing here ❤️

[johan-bjareholt]

Fixed

aw-client: 4b11b94

aw-server: ActivityWatch/aw-server@3d841d9