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

improve formulas and revive tests #700

Merged
merged 2 commits into from
Oct 9, 2014

Conversation

kleinschmidt
Copy link
Contributor

This implements a couple of missing features for formulas, most notably three-way and higher interactions and proper handling of associative and distributive properties of the :+ and :& operators (open to suggestions on how to improve any of these). It also restores the formula tests, removing tests for features that were previously purged and updating the remaining tests to use the up-to-date DataFrames interface.

@johnmyleswhite
Copy link
Contributor

I'll look right now. This is a very impressive PR from the look of it.

@johnmyleswhite
Copy link
Contributor

LGTM. Maybe squash this before merging?

@johnmyleswhite
Copy link
Contributor

@kleinschmidt, I added you as a collaborator to JuliaStats so that you'd be a collaborator on this repo.

…nd distribute rules in formulas, and R-compatible ordering of interaction terms after sorting
@kleinschmidt
Copy link
Contributor Author

Okay, I've squashed these (although that appears to have killed @dmbates comment). Apologies for my github ignorance but I don't see how to actually merge this (or perhaps I'm not actually empowered to do that?).

@johnmyleswhite
Copy link
Contributor

Did you accept the collaborator invite? If so, you should have a "merge pull request" section right above the comments box. It's yellow right now because Travis is still running.

@johnmyleswhite
Copy link
Contributor

I believe @dmbates's comment lives on in GitHub's internal history as a "comment on an outdated diff".

@kleinschmidt
Copy link
Contributor Author

Ah, right, missed that. I'll merge this now.

kleinschmidt added a commit that referenced this pull request Oct 9, 2014
improve formulas and revive tests
@kleinschmidt kleinschmidt merged commit 6ccf483 into JuliaData:master Oct 9, 2014
nalimilan pushed a commit that referenced this pull request May 26, 2022
improve formulas and revive tests
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