search

DSGT-DLP / Deep-Learning-Playground

Web Application where people new to Deep Learning can input a dataset and toy around with basic Pytorch modules without writing any code
MIT License
26 stars 8 forks source link

Added LearnModules to Nextjs #843

Closed Dashrekker closed 1 year ago

Dashrekker commented 1 year ago

Added LearnModules to NextJs. The previous exercise implementation doesn't exercise anymore, so I just used trainspace. image image image

karkir0003 commented 1 year ago

u can mark pr as ready for review when you are ready? @Dashrekker

karkir0003 commented 1 year ago

did u test this @Dashrekker

Dashrekker commented 1 year ago

@karkir0003 It works with the exception of user progress, which I don't think has been implemented yet.

karkir0003 commented 1 year ago

hmm. i see. we are thinking of revamping learning modules to be more project oriented. talk to daniel, surya, faris for more detail

farisdurrani commented 1 year ago

lgtm, fix the conflicts and await for either Karthik or Daniel's green light before you merge. It would be helpful to add a screen recording but that's not necessary

karkir0003 commented 1 year ago

@farisdurrani i think we need to rethink the whole learning modules design

farisdurrani commented 1 year ago

Agree @karkir0003 but I think this PR is still worthwhile to merge in. It has some components we'll need for the project-oriented version, especially the trainspace exercise, and it gives an initial framework for the next developers to work on

karkir0003 commented 1 year ago

ok. agreed @farisdurrani

karkir0003 commented 1 year ago

merge when youre ready

farisdurrani commented 1 year ago

Thanks Karthik, @Dashrekker please fix the conflicts and then proceed with merging

farisdurrani commented 1 year ago

You can press merge now @Dashrekker . Thanks!