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

add PoC for an experimental math sugar module #12

Merged
merged 6 commits into from
Jun 16, 2024
Merged

Conversation

Vindaar
Copy link
Member

@Vindaar Vindaar commented Jan 23, 2022

This is only a draft, as obvious from the code.

I think the idea has some merit. Implementations here aren't very sophisticated (i.e. the macros need error handling etc.). It would be nice to have imo.

@Clonkk
Copy link
Member

Clonkk commented Jun 12, 2024

@Vindaar Let's not have stale PR :) Do you think this is mergeable ?

@Vindaar
Copy link
Member Author

Vindaar commented Jun 13, 2024

I just have to clean out the comments etc. and then we can merge it I guess. I don't think it will really see use, but why not have an experimental sugar module 🤭

@Vindaar Vindaar marked this pull request as ready for review June 16, 2024 16:39
@Vindaar Vindaar force-pushed the experimentalSugar branch from 69843fb to 5963a5a Compare June 16, 2024 16:39
@Vindaar
Copy link
Member Author

Vindaar commented Jun 16, 2024

CI failure the same as for PR #6. Will be fixed in arraymancer.

@Vindaar Vindaar merged commit 201ddcb into main Jun 16, 2024
0 of 9 checks passed
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