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 audio device selection to meetings #561

Merged

Conversation

EzraEllette
Copy link
Contributor

description

Allows users to select the audio devices for a meeting.
Does not affect summary or person identification.
related issue: #553

type of change

  • bug fix
  • new feature
  • breaking change
  • documentation update

how to test

  1. check / uncheck devices
  2. verify transcript change

checklist

  • i have read the CONTRIBUTING.md file
  • i have added the custom cursor AI prompt to my settings as mentioned in CONTRIBUTING.md and used to write this PR
  • my code follows the project's style guidelines
  • i have performed a self-review of my code
  • i have updated the documentation if necessary
  • my changes generate no new warnings
  • i have added tests that prove my fix is effective or that my feature works
  • all tests pass locally with my changes
    /claim 553

Copy link

vercel bot commented Oct 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
screenpipe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 22, 2024 5:37pm

@louis030195
Copy link
Collaborator

Screenshot 2024-10-22 at 08 16 02

how does your page looks like?

@louis030195
Copy link
Collaborator

Screenshot 2024-10-22 at 08 17 37

not sure what happened i only have 1 and 18 (even on main) -

@@ -267,7 +279,9 @@ export default function MeetingHistory() {
},
{
role: "user" as const,
content: `${enhancedPrompt}:\n\n${meeting.fullTranscription}`,
content: `${enhancedPrompt}:\n\n${meeting.segments
Copy link
Collaborator

Choose a reason for hiding this comment

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

i think it's important to keep the timestamp (probably in a more human readable format) both for human and machine summary

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay I'll update that

Copy link
Collaborator

Choose a reason for hiding this comment

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

image eg this but more human readable date and probably should tell the LLM current date and stuff about local / UTC or timezone if useful

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll make sure it is the same format

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

@louis030195
Copy link
Collaborator

/approve

Copy link

algora-pbc bot commented Oct 22, 2024

@louis030195: The claim has been successfully added to reward-all. You can visit your dashboard to complete the payment.

@louis030195 louis030195 merged commit fc43cab into mediar-ai:main Oct 22, 2024
3 of 7 checks passed
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