-
Notifications
You must be signed in to change notification settings - Fork 39
Implement formulaparser.m
#382
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
Conversation
Removed the entire FormulaParser function and its helper functions.
|
I think Can you add support for |
|
@pr0m1th3as yes sure!! will come up with the changes. |
|
you opened this PR without referencing the roadmap we established in Issue #266 or the ongoing discussion in #363. Since I am actively working on the |
The PR #363 is stalled because I explicitly asked you to stop pouring AI slop into that PR, but you kept going in the same direction. I haven't had the time to update the contribution guidelines yet, but let me be crystal clear about. I will not accept or tolerate any kind of contribution in any of the packages I maintain if I see any signs that it is the result of vibe coding. |
|
Hello @pr0m1th3as the changes have now been completed, firstly since "-" was already deletion operation and commas in LHS were not processed, now the formula is split into LHS and RHS and processed likewise. I also aded a new helper function to create list of the variables on the LHS side and the RHS side is processed as per the previous implementation but as |
442932e to
de939d8
Compare
though a previous initiative was taken to build this parser , #363
Here is a refined
formulaparser.mthat does support the fit models and anova function.This is what has been implemented in short :
Demos, documentation, BISTs have been added lately.