johko / computer-vision-course

This repo is the homebase of a community driven course on Computer Vision with Neural Networks. Feel free to join us on the Hugging Face discord: hf.co/join/discord
MIT License
371 stars 123 forks source link

Unit 3 : Vision Transformers / Transfer Learning & Fine-Tuning Chapter Content #204

Closed shreydan closed 3 months ago

shreydan commented 4 months ago

hello, this PR adds the following chapter content for Unit 3: Vision Transformers : Transfer Learning and Fine-Tuning

Contributions:

@asusevski

@hanouticelina

@shreydan

@sezan92

@Anindyadeep

review-notebook-app[bot] commented 4 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

asusevski commented 4 months ago

Thanks a lot! mostly left nits. it'd be great for upcoming PRs to be atomic since it's hard to review multiple pages coming from different contributors repetitively iterating on them

Very sorry about that @merveenoyan !! We were just concerned about timelines, that we wouldn't get all of our individual changes merged in time. We realize this must have been super annoying to go through!

MKhalusova commented 4 months ago

Please make sure to fix grammatical mistakes before merging.

SolshineCode commented 4 months ago

Looks good to me 👍🏽 (granted noted grammatical correction occur)

Anindyadeep commented 4 months ago

Hi I am adding the reference for PR #218 where I added the mdx file for fine-tuning for object detection.

merveenoyan commented 4 months ago

@shreydan @asusevski I think the table of contents is not up to date, can you merge the main of this repository into your branch? then we can merge

shreydan commented 4 months ago

@merveenoyan merged and update table of contents! Thank you!

merveenoyan commented 4 months ago

@asusevski can you approve so we can merge? I'll fix CI afterwards, the previously merged PR broke that

merveenoyan commented 3 months ago

@asusevski @shreydan I fixed a bunch of LaTex errors and more, once they're merged, you can merge the main branch into this one and we can merge this one OR if you don't want to wait, feel free to incorporate following changes: https://github.com/johko/computer-vision-course/commit/8b410ff9b0210a0dbe86f540a040141f103e22c3 https://github.com/johko/computer-vision-course/commit/8b410ff9b0210a0dbe86f540a040141f103e22c3 https://github.com/johko/computer-vision-course/commit/bd5673b36b8b75a39e941a211ba10c023c349399 https://github.com/johko/computer-vision-course/commit/63cabaa80138bc3128b00163980576d58a19ce8f

merveenoyan commented 3 months ago

@asusevski I fixed bunch of grammar errors and made the CI green, if you can approve we can merge

merveenoyan commented 3 months ago

@asusevski had to solve merge conflict, sorry, can you approve again?