-
Notifications
You must be signed in to change notification settings - Fork 11
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
readme #51
Comments
Ha no problem. Totally fair. Criticism is the point. Will fix this and the
|
I went ahead and removed it. I'll reimplement later if needed. |
|
do you mean references to articles? or links to the packages? |
References to articles. E.g. if you look at
|
ok did those things. thanks |
You have two times
extract_proximity
in the readme. Moreover you haverandomforest_distance
, but the function israndomforest_dist
.You do not mention
randomforest_dist
in the paper, but maybe this is not a problem. I really do not know what happens inrandomforest_dist
, you could add a reference or better explanation in the help.Also references in the help of the other functions.
Sorry for being very critical, but I think this could really improve the documentation.
The text was updated successfully, but these errors were encountered: