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

mlv_lite and mlv_rec are highly redundant, should be merged #113

Open
reticulatedpines opened this issue Aug 21, 2023 · 0 comments
Open

mlv_lite and mlv_rec are highly redundant, should be merged #113

reticulatedpines opened this issue Aug 21, 2023 · 0 comments

Comments

@reticulatedpines
Copy link
Owner

Both mlv_lite and mlv_rec perform very similar functions. Functionality should be merged while keeping acceptable performance. If necessary, optional expensive functionality could be behind a toggle in a menu (metadata per frame?).

First need to determine required functionality (mlv_rec does more, except, mlv_lite does lossless compression... anything else it's better at?). Card spanning is important. Both should do sound, need to confirm this.

Some discussion of differences here: https://discord.com/channels/671072748985909258/671072748985909264/1142858075682906132
reticulatedpines added a commit that referenced this issue Aug 21, 2023
@reticulatedpines
modules: only autoload when unambiguous
a9b536a 
If multiple modules provide the same symbol,
don't autoload any of them.  Annoyingly, this
prevents all autoloading, because current status
of module system is a bit janky.

See:
#112
#113

As we clean up clashing symbol names and redundant code,
autoloading will start to work properly.
@reticulatedpines reticulatedpines mentioned this issue Dec 20, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@reticulatedpines