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
370 stars 123 forks source link

Updates links to images and fixes a link to a Gradio space that was getting a 404 #219

Closed kfahn22 closed 4 months ago

kfahn22 commented 4 months ago

Updates links to images: chapters/en/Unit 4 - Multimodal Models/CLIP and relatives/Introduction.mdx chapters/en/Unit 4 - Multimodal Models/CLIP and relatives/losses.mdx chapters/en/Unit 6 - Basic CV Tasks/segmentation.mdx chapters/en/Unit 8 - 3D Vision, Scene Rendering and Reconstruction/nerf.mdx chapters/en/Unit 9 - Model Optimization/intro_to_model_optimization.mdx chapters/en/Unit 12 - Ethics and Biases/ethics-bias-ai.mdx

Update link to Gradio space: chapters/en/Unit 4 - Multimodal Models/CLIP and relatives/blip.mdx

kfahn22 commented 4 months ago

In updating the image links, I noticed that the LateX is formatted properly in line 18 of chapters/en/Unit 4 - Multimodal Models/CLIP and relatives/losses.mdx, but not within text of next paragraph. I tried updating by adding an extra $ but that didn't fix it. How can this be addressed?

Another TODO: links to notebooks need to be fixed

kfahn22 commented 4 months ago

@MKhalusova

This is what is looks like in pr #212:

Screenshot 2024-02-29 at 1 30 34 PM

I edited it to $$ so it looks a little different in this pull request, but still not rendering properly.

MKhalusova commented 4 months ago

@kfahn22 This is odd. I'm not sure exactly why this happens, but one guess that I have is that list formatting might be messing with it. As an experiment, can you make it a paragraph instead of a bulleted list to see if this will help?

kfahn22 commented 4 months ago

@MKhalusova

It didn't fix it.

Screenshot 2024-02-29 at 1 52 34 PM
MKhalusova commented 4 months ago

@kfahn22 I think I found the issue - https://github.com/huggingface/doc-builder#latex The inline latex is expected to be in the following format: \\( ... )\\

kfahn22 commented 4 months ago

@MKhalusova The LateX is rendering, but the spacing is a little off (I noticed the same thing on the reference page, though).

Screenshot 2024-02-29 at 5 37 03 PM
kfahn22 commented 4 months ago

I have experimenting with adding spaces, but that has not been productive. Ironically, in a couple of tries, the spacing for some of the inline Latex has been OK (but never all of them), and I can't figure out why.

I also just realized in looking through the mdx files (looking to see if there was an example of inline LateX without the spacing issue) that there are additional mdx files with image links that need to be updated. They aren't in the course assets folder--dooes not matter?

merveenoyan commented 4 months ago

pinging @mishig25 here he likely knows

merveenoyan commented 4 months ago

@kfahn22 in BLIP demo you can replace hysts with merve and it will work, I forked that demo and fixed previously

kfahn22 commented 4 months ago

@merveenoyan I updated the link from "https://hysts-BLIP2-with-transformers.hf.space" to "https://hysts-BLIP2.hf.space" and I just checked and it is working. Should I still update?

merveenoyan commented 4 months ago

@kfahn22 BTW I'm fixing inline LaTeX rn let's see

kfahn22 commented 4 months ago

@merveenoyan

FYI, I tried the notebook link in chapters/en/Unit 9 - Model Optimization/tools_and_frameworks.mdx and got Access Denied message. (And the links to notebooks in mdx files I added won't work.)

Also, I just noticed that the one of the images in chapters/en/Unit 13 - Outlook/hyena.mdx isn't in the course-assets repo. (toeplitz_conv.jpg)

kfahn22 commented 4 months ago

@merveenoyan I am closing this PR since it looks like #224 addresses almost all of the same issues, except: chapters/en/Unit 8 - 3D Vision, Scene Rendering and Reconstruction/nerf.mdx chapters/en/Unit 9 - Model Optimization/intro_to_model_optimization.mdx