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.

Sign up for GitHub

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

Jump to bottom

[bug] memory leak (audio related?) #1269

Open
louis030195 opened this issue Feb 4, 2025 · 4 comments
Open

[bug] memory leak (audio related?) #1269

louis030195 opened this issue Feb 4, 2025 · 4 comments
Labels
💎 Bounty bug Something isn't working

Comments

@louis030195
Copy link
Collaborator

related

07cdb82

7d31420

i think it was introduced by my massive changes on the core codebase that was necessary to fix the multi monitor poor experience

probably some data structure getting filled too much, or maybe overusing Arc getting circular references or something like this
@louis030195 louis030195 added the bug Something isn't working label Feb 4, 2025
@louis030195 louis030195 changed the title [bug] memory leak [bug] memory leak (audio related?) Feb 4, 2025
@louis030195
Copy link
Collaborator Author

update: i have local code likely fixed the issue, testing more

louis030195 added a commit that referenced this issue Feb 5, 2025
@louis030195
fix: memory leak #1269, document how to use tokio-console, refactor: …
bb5232d 
…improve obsidian prompt to link people using obsidian syntax, refactor: improve ux to display user is logged in, refactor: make share log use the backend log instead of app by default, fix: #1202
@louis030195
Copy link
Collaborator Author

louis030195 commented Feb 5, 2025 •
edited
Loading

still think there is an issue

/bounty 200

@algora-pbc Algora PBC
Copy link

algora-pbc bot commented Feb 5, 2025 •
edited
Loading

💎 $200 bounty • Screenpi.pe

Steps to solve:

  1. Start working: Comment /attempt #1269 with your implementation plan
  2. Submit work: Create a pull request including /claim #1269 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to mediar-ai/screenpipe!

Add a bounty • Share on socials

Attempt Started (GMT+0) Solution
🟢 @EzraEllette Feb 5, 2025, 1:44:06 AM WIP

@EzraEllette
Copy link
Contributor

EzraEllette commented Feb 5, 2025 •
edited by algora-pbc bot
Loading

/attempt #1269

Algora profile Completed bounties Tech Active attempts Options
@EzraEllette 23 mediar-ai bounties
TypeScript, Rust,
JavaScript & more
﹟1185
 Cancel attempt

louis030195 added a commit that referenced this issue Feb 5, 2025
@louis030195
chore: use weak references #1269
c7e278a
@sourcery-ai sourcery-ai bot mentioned this issue Feb 5, 2025
Merged
louis030195 added a commit that referenced this issue Feb 5, 2025
@louis030195
fix: some improvement on memory, still ongoing #1269, feat: make it p…
453e741 
…ossible to full screen pipe, focused by default,
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
💎 Bounty bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@louis030195 @EzraEllette