Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add more watchers/importers #137

Merged
merged 2 commits into from
Sep 21, 2024
Merged

Add more watchers/importers #137

merged 2 commits into from
Sep 21, 2024

Conversation

RTnhN
Copy link
Contributor

@RTnhN RTnhN commented Sep 6, 2024

This adds three new watchers/importers to the docs:

Lastfm importer

This simply imports the last.fm data into AW by watching a folder. Since it does not include song duration on the export, I just set the default time to 60 seconds. I think this is maybe even a little more consistent in recording song plays than the Spotify watcher since the Spotify watcher sometimes misses songs.

Lifecycle importer

Lifecycle is a great iOS app that tracks location and activity. It also has a really nice export. This is particularly helpful to see what you were doing when you are away from your computer that has AW. This also watches a folder for changes.

Buttons watcher

For work related stuff, I have a couple "buckets" that I need to file hours into. While I could just determine that from data in AW, it is nice to just have a clear label to make it obvious what "state" I am in as a check. It is also really satisfying to press a physical button to change "state". The light up buttons also are a great indicator about what "state" I should be in. The hardware is just an Arduino that talks to the computer though serial. The watcher includes the Arduino code although I might need to add a diagram on how to wire up the buttons to the Arduino (it is really pretty simple though).

This also fixes the build by updating the Upload Artifact to the newest version.


🚀 This description was created by Ellipsis for commit 9e96412

Summary:

Added Lastfm, Lifecycle importers, and Buttons watcher to documentation for tracking music, location, and work states.

Key points:

  • Added aw-importer-lastfm to src/importers.rst for importing Last.fm data.
  • Added aw-importer-lifecycle to src/importers.rst for importing Lifecycle app data.
  • Added aw-watcher-buttons to src/watchers.rst for tracking work states using Arduino hardware.

Generated with ❤️ by ellipsis.dev

Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to 9e96412 in 7 seconds

More details
  • Looked at 27 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. src/importers.rst:7
  • Draft comment:
    Add a period after 'data'.
- :gh-aw:`aw-importer-smartertime`, imports from `smartertime`_. Useful for importing Android screentime data. (Note ActivityWatch also has an Android app :gh-aw:`aw-android`)
  • Reason this comment was not posted:
    Confidence changes required: 10%
    The PR adds new watchers and importers to the documentation. The format for the new entries is consistent with existing entries, but there is a missing period in one of the descriptions.

Workflow ID: wflow_kz0RXdMGUKJBby4Q


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@RTnhN
Copy link
Contributor Author

RTnhN commented Sep 20, 2024

@ErikBjare Hey Erik, was there anything else that this needed? If you don't want to merge it or if you were busy with other things, that is totally fine. I just wanted to make sure that there was not a disconnect.

@ErikBjare
Copy link
Member

It's great! Merging.

Sorry for the delay and thanks for the ping.

@ErikBjare ErikBjare merged commit 47a534f into ActivityWatch:master Sep 21, 2024
1 check passed
@RTnhN
Copy link
Contributor Author

RTnhN commented Sep 21, 2024

It's great! Merging.

Sorry for the delay and thanks for the ping.

No problem. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants