Parasgupta44 / ml-resources-react

A simple react app displaying resources to learn Machine learning.
https://sharp-gates-258dae.netlify.app/
MIT License
14 stars 25 forks source link

Add new resources to learn machine learning in the React app. #1

Open Parasgupta44 opened 4 years ago

Parasgupta44 commented 4 years ago

Task

Add new resources to learn machine learning from. Please check Readme.md and Contributing.md in the repo's root for more details.

supanthapaul commented 4 years ago

Added new resources from mathematicalmonk and David Silver. Please review the PR.

12

Parasgupta44 commented 4 years ago

Added new resources from mathematicalmonk and David Silver. Please review the PR.

12

Merged for deployment to master ! Thanks.

CiTRuS93 commented 3 years ago

Maybe we can automate this with some kind of form?

Parasgupta44 commented 3 years ago

Yeah...that's a nice idea. So, do you have any ideas how should we go for it because my plan was to keep the resources content as static as possible (that's why no db), so that reviewing them for sanity would be much easier !

CiTRuS93 commented 3 years ago

Maybe writing into some "suggestions.json" and review before adding to resources.json?

Parasgupta44 commented 3 years ago

Maybe writing into some "suggestions.json" and review before adding to resources.json?

Yeah, that seems like a good option.

I guess db setup n stuff can be an overkill for this (?) Or maybe not much of a hassle ?

CiTRuS93 commented 3 years ago

I don't think we really need a db cause it's only one table. The bigger issue is how to prevent spam in this form.

Parasgupta44 commented 3 years ago

The bigger issue is how to prevent spam in this form.

Yeah, this one is crucial. Will look into this for sure !!