Two separate images directories #48

Closed
opened 2026-03-22 15:23:20 -05:00 by GiteaMirror · 3 comments
Owner

Originally created by @V0XNIHILI on GitHub (Dec 10, 2023).

Originally assigned to: @V0XNIHILI on GitHub.

Currently, there are two different image directories in the root of this project:

image

@profvjreddi would you be okay if I move the images_ml_frameworks directory into the main images folder?

Originally created by @V0XNIHILI on GitHub (Dec 10, 2023). Originally assigned to: @V0XNIHILI on GitHub. Currently, there are two different image directories in the root of this project: <img width="341" alt="image" src="https://github.com/harvard-edge/cs249r_book/assets/24796206/3680e70d-5de4-4354-aeed-8a45878fc061"> @profvjreddi would you be okay if I move the `images_ml_frameworks` directory into the main `images` folder?
GiteaMirror added the area: tools label 2026-03-22 15:23:20 -05:00
Author
Owner

@profvjreddi commented on GitHub (Dec 10, 2023):

Hi @V0XNIHILI - yes, please go ahead and do that.

@profvjreddi commented on GitHub (Dec 10, 2023): Hi @V0XNIHILI - yes, please go ahead and do that.
Author
Owner

@profvjreddi commented on GitHub (Dec 10, 2023):

One thing we should do is to create a chapter for each folder and keep all relevant content under that folder.

E.g. optimizations.qmd goes into optimizations/optimizations.qmd and there is a folder for images like optimizations/images/.

We have an issue with that https://github.com/harvard-edge/cs249r_book/issues/22

@profvjreddi commented on GitHub (Dec 10, 2023): One thing we should do is to create a chapter for each folder and keep all relevant content under that folder. E.g. `optimizations.qmd` goes into `optimizations/optimizations.qmd` and there is a folder for images like `optimizations/images/`. We have an issue with that https://github.com/harvard-edge/cs249r_book/issues/22
Author
Owner

@profvjreddi commented on GitHub (Dec 10, 2023):

This is addressed in the big merge I did here: 3625af9cc4

@profvjreddi commented on GitHub (Dec 10, 2023): This is addressed in the big merge I did here: 3625af9cc4bfbc5e4f6e1eb9272b11d008825164
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/cs249r_book#48