Kyle Gilde
Kyle Gilde
Should the set of features be a clique, where they are all correlated with each other? https://en.wikipedia.org/wiki/Clique_(graph_theory)
What if we added an argument to the class that checked and retained only the features in the group that are correlated to all the other features at the specified...
@solegalli This code needs unit tests, but I wanted to get feedback on this newer implementation of the class to see if it aligns with your expectations.
It looks like the failures are happening due to an unrelated boxcox failure
> No problem! I hope you had a great vacation!
I need to tidy the files and commits, and I need to create real unit tests. I will have time this weekend I believe.
@solegalli I think that I removed the unrelated files. I just need to find time to create unit tests.
@solegalli , I followed your instructions, and the checks ran again, but they failed for the same reasons, which are unrelated to my code changes. What should I try next?
I merged your PR. I see your commit in the commit history, but we are getting errors. I tried following your Docs instructions one more time, but it did not...
I didn't do my last commit correctly and I really messed up my file :(