ml-resources-react icon indicating copy to clipboard operation
ml-resources-react copied to clipboard

Published 20 hours ago • verified publisher icon Parasgupta44

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

Open Parasgupta44 opened this issue 4 years ago • 8 comments

Task

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

Parasgupta44 avatar Sep 29 '20 19:09 Parasgupta44

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

supanthapaul avatar Sep 30 '20 19:09 supanthapaul

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

Merged for deployment to master ! Thanks.

Parasgupta44 avatar Sep 30 '20 20:09 Parasgupta44

Maybe we can automate this with some kind of form?

CiTRuS93 avatar Oct 02 '20 10:10 CiTRuS93

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 !

Parasgupta44 avatar Oct 02 '20 11:10 Parasgupta44

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

CiTRuS93 avatar Oct 02 '20 11:10 CiTRuS93

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 ?

Parasgupta44 avatar Oct 02 '20 11:10 Parasgupta44

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.

CiTRuS93 avatar Oct 04 '20 08:10 CiTRuS93

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

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

Parasgupta44 avatar Oct 04 '20 23:10 Parasgupta44