KNN section work about data-science-toolkit HOT 10 CLOSED

@pmaji There is more in my repo now, I will add to yours soon.

from data-science-toolkit.

@pmaji Alright, I have it how I want it I think, I've addressed most of your suggestions (it is in md and I have included an example including a benchmark comparing a library implementation to my version).

Do you have any suggestions for better visuals? I am open to ideas if you've seen something designed for this before (I know you've mentioned it, but if you could be more specific). I realize my visuals aren't as strong as they could be.

from data-science-toolkit.

Ah okay I think I understand, I had to do something similar anyways to make the md's I posted so I understand, but now I see the point of doing that as opposed to just making the md's.

from data-science-toolkit.

I have been busy lately, going to work more on it soon!

from data-science-toolkit.

@TWilliamBell Fantastic! Looking forward to the collaboration 👍

from data-science-toolkit.

@TWilliamBell This is awesome work! I have one main suggested change that is a high-level formatting proposal. Presently, your markdowns are rendered as straight HTML, which is actually what keeps the plots, etc., from displayed in browser when you view the markdown.

If you check out any of my Rmd's -- say the logit one for example -- you'll see I have a few options there that make it tailored to display via Github. I've pasted the crux of the code below. If you transform your Rmds into that format and submit a new PR, it'll add a files folder, override the old MDs, and all your plots will display :). See below for the header / YAML code that will do the trick for GitHub markdowns. (sadly the pasted code loses its indent structure--which is necessary to get the code to run. As such, I'd recommend just going to any of my markdowns and copying the output: call.

`---

title: "Logistic Regression"
author: Paul Jeffries
output:
github_document:
toc: TRUE
toc_depth: 2

---`

Finally, the other thing I might do is submit some formatting changes to the main markdown--just to be able to lay out in a slightly more seamless way what sort of progressions folks should go through via the various Rmds you have. Those are edits I can make later though :)

This is a truly awesome start! Once the plots are actually displaying via the fix I suggested I can think through what other viz's might be appropriate.

from data-science-toolkit.

I had your thought process in mind because the Rmds don't work on GitHub, but I thought it was good enough that I had the mds display it in a proper literate programming code-visuals-text manner. What do you think is the benefit of the changes over the mds already there?

I'm sorry if you think you've already mentioned it but I might have missed it.

from data-science-toolkit.

@TWilliamBell sure thing! I probably wasn't clear--that's on me :)

What i mean is that changing the output type of your markdowns will actually mean that folks will be able to see your outputs and plots on github--presently we can't see any of your plots sadly.

To add more clarity to my previous comment about what to change, it's just one line of code and a re-render. Check out the explanation at the link below and let me know if that makes sense.

https://rmarkdown.rstudio.com/github_document_format.html

from data-science-toolkit.

Additional explanation of what I mean here on stack too:

https://stackoverflow.com/questions/39814916/how-can-i-see-output-of-rmd-in-github

from data-science-toolkit.

Hey @TWilliamBell -- I was doing some repo maintenance and I never realized that you went back and made the formatting changes I requested, so now all the visuals are viewable on GitHub! This is awesome!

Does this repo here represent the most recent version of your work? If so, I'll fork it shortly and add it into my repository here, cross-linking to your repo.

Excited to add in your contributions!

from data-science-toolkit.