Skip to content

fix: fix memory intensive crate + misc - #19

Closed
0xbrayo wants to merge 3 commits into
2e3s:mainfrom
0xbrayo:main
Closed

fix: fix memory intensive crate + misc#19
0xbrayo wants to merge 3 commits into
2e3s:mainfrom
0xbrayo:main

Conversation

@0xbrayo

@0xbrayo 0xbrayo commented May 27, 2026

Copy link
Copy Markdown

The macos binary used a lot of memory, >500MB sometimes. There was a memory inefficient crate, media-remote that was responsible. Patched it, it depended on a image crate that consumed a lot of memory.

@2e3s

2e3s commented May 29, 2026

Copy link
Copy Markdown
Owner

Thank you, I'll test it, will you propose this fix to media-remote?
At the very least, if they have such a troublesome dependency, it may need to be a switchable feature, even if enabled by default.

@2e3s

2e3s commented May 29, 2026

Copy link
Copy Markdown
Owner

https://www.reddit.com/r/rust/comments/irgb4u/imagers_and_images_with_hugeish_dimensions/
It seems to be somewhat expected for image library, while unexpected for media-remote, I think something is off. Maybe, most likely, the library or the way it loads is a massive overkill.

@0xbrayo

0xbrayo commented May 29, 2026

Copy link
Copy Markdown
Author

Thank you, I'll test it, will you propose this fix to media-remote? At the very least, if they have such a troublesome dependency, it may need to be a switchable feature, even if enabled by default.

I will, was just making sure I didn't miss or accidentally break something.

@0xbrayo

0xbrayo commented Jun 9, 2026

Copy link
Copy Markdown
Author

Fix merged upstream in media-remote nohackjustnoobb/media-remote#13, you can bump the version number to get the fix

@0xbrayo 0xbrayo closed this Jun 9, 2026
@2e3s

2e3s commented Jul 4, 2026

Copy link
Copy Markdown
Owner

Awesome, thanks @0xbrayo for pushing through this improvement 🚀
I'll bump and release 1-2 weeks later as soon as I can

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