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

[RFC] Add a link to MacroLibX #65

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xtrm-en
Copy link

@xtrm-en xtrm-en commented Nov 20, 2023

Repo link: https://github.com/seekrs/MacroLibX/

MacroLibX is yet another MLX rewrite by 42Angoulême students, using SDL and Vulkan.

Its main difference with https://github.com/codam-coding-college/MLX42/ is that it aims in keeping the same API and tries to be a drop-in replacement, with only greater performance in added benefits.

It's still in its early stages, so I'm starting this issue as a draft and as a Request For Comments on both:

  • if it should be included in the extension, as I think diversity in MLX implementations would benefit people the most
  • if not, what would be the improvements necessary for its inclusion (if any)?

@FreekBes
Copy link
Owner

Hello,

I like the part where it aims to be a drop-in replacement to the standard minilibx.

Diversifying the choice in MLX libraries was never really the intention of the addition of MLX42. The requirement for adding a link to a project like this would be that the project must be accepted and deemed helpful by the staff of a handful of 42 campuses from all over the world. Both MLX42 and Find Peers, projects that are currently featured by Improved Intra, meet these requirements.

I suggest fleshing out your repository a tad bit more, getting rid of bugs and finding support from students, then come back to me and we can discuss more.

@FreekBes FreekBes added feature request New feature or request question Question or request of information to developers labels Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request question Question or request of information to developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants