Skip to content

Conversation

vishwamartur
Copy link

@vishwamartur vishwamartur commented Jan 20, 2025

Related to #766

Add support for multiple monitors and provide screenshots of the visualization containing more than one monitor.

  • openadapt/utils.py

    • Update take_screenshot function to iterate through all monitors and capture screenshots.
    • Add a new function take_screenshots to return a list of screenshots from all monitors.
  • openadapt/visualize.py

    • Modify main function to handle and display screenshots from multiple monitors.
    • Update the HTML generation logic to include screenshots from all monitors.
  • openadapt/record.py

    • Update read_screen_events function to support capturing screenshots from multiple monitors.
  • tests/openadapt/test_utils.py

    • Add unit tests for the new take_screenshots function.

/claim #766

Related to OpenAdaptAI#766

Add support for multiple monitors and provide screenshots of the visualization containing more than one monitor.

* **openadapt/utils.py**
  - Update `take_screenshot` function to iterate through all monitors and capture screenshots.
  - Add a new function `take_screenshots` to return a list of screenshots from all monitors.

* **openadapt/visualize.py**
  - Modify `main` function to handle and display screenshots from multiple monitors.
  - Update the HTML generation logic to include screenshots from all monitors.

* **openadapt/record.py**
  - Update `read_screen_events` function to support capturing screenshots from multiple monitors.

* **tests/openadapt/test_utils.py**
  - Add unit tests for the new `take_screenshots` function.
Copy link

algora-pbc bot commented Jan 20, 2025

💵 To receive payouts, sign up on Algora, link your Github account and connect with Stripe.

@abrichr
Copy link
Member

abrichr commented Jan 20, 2025

Thank you @vishwamartur! Can you please paste some screenshots of visualize.py and the dashboard? Also can you please upload the HTML file produced with visualize.py --cleanup=False?

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

Successfully merging this pull request may close these issues.

2 participants