-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Idea: use MacroTools.jl? #149
Comments
The code is Once, I tried to add something to MacroTools, but it never got through. I am not interested in adding a dependency for a project that is impossible to update. |
Aha, I should have done a better search! Thank you for the background and reasoning. |
You're welcome. A simplified version of this parsing code exist at https://github.com/aminya/VarStructs.jl/blob/master/src/parse.jl I might be able to refactor
|
That's an interesting package that I had not yet seen, thank you for the pointer! |
I know at this point it could be a heroic effort, but after looking at the parser for
@aml
, I thought it would be appropriate to suggest usage of MacroTools.jl and their@capture
macro. It might be the biggest help around here:AcuteML.jl/src/@aml/@aml_parse.jl
Lines 51 to 389 in 6b87580
The text was updated successfully, but these errors were encountered: