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

Ux analytics plugin #3044

Draft
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

daykin
Copy link
Collaborator

@daykin daykin commented Jun 13, 2024

No description provided.

@kasemir
Copy link
Collaborator

kasemir commented Jun 13, 2024

#3046 added the essential hooks into the display runtime.

The data collection and analysis are somewhat separate.
Most end-users who download phoebus to run on their office PC don't need it.
It meant to enabled in a control room, requires something like mongo for storage etc.

Still, it's currently not much code, so once a Readme.md is added that shows what to prepare, how to enable it, and maybe one example for looking at the data, maybe we do include it in this repo?

@daykin daykin force-pushed the ux-analytics-plugin branch from 6843e25 to 6738310 Compare June 14, 2024 14:39
@shroffk
Copy link
Member

shroffk commented Jun 17, 2024

@daykin could you create a readme.md using the material you presented during the codeathon?

@daykin daykin added the wip work in progress label Jun 17, 2024
@daykin
Copy link
Collaborator Author

daykin commented Jun 17, 2024

Yes, I will add something once development is a bit more stable. Still hammering out the strategy for "How was this page opened" (From file browser? Menu entry? Action button?)

@daykin daykin marked this pull request as draft June 19, 2024 13:54
@daykin daykin force-pushed the ux-analytics-plugin branch from b4e9263 to 06ad320 Compare June 19, 2024 21:43
@daykin
Copy link
Collaborator Author

daykin commented Jul 2, 2024

To do list:

  • Use Credential manager instead of hand-rolled thingy
  • Write tests
  • Eliminate separate pathways for action button vs other opening sources?
  • Heat mapping

@daykin daykin force-pushed the ux-analytics-plugin branch from e1e9f81 to 7753c37 Compare December 12, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wip work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants