Update findings and recommendations for storage optimization - #12
Open
kameron-ctrl wants to merge 1 commit into
Open
Update findings and recommendations for storage optimization#12kameron-ctrl wants to merge 1 commit into
kameron-ctrl wants to merge 1 commit into
Conversation
Updated the findings and recommendations section regarding offline reference functionality, file compression, and search feature considerations.
SilasVM
requested changes
Aug 5, 2026
| - [Kam will need help with this] - What we recommend next and why | ||
| - Good place to mention what would be a reasonable size for the .zip | ||
| - Also a good place to share thoughts on including all images, search functionality, language support, etc. | ||
| - For a smaller downloadable zip we must discuss what all we want in the zip. The search feature is expendable especiallly since the filter feature already exists. Also the assets is taking up a great |
Contributor
There was a problem hiding this comment.
Great work and investigation into the further viability of ffmpeg Kam! Earlier in the document @riahdollxo suggested leaving the search function in because despite it's lack of functionality the size is negligible enough to make removing it more of a hassle than the problems that it may cause. I'll leave it up to you to decide which route is best, perhaps after a brief discussion with @riahdollxo . I just would want to make sure both this section on line 47 and the section on line 38 present a united front on the suggested handling of the search function.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated the findings and recommendations section regarding offline reference functionality, file compression, and search feature considerations.