Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Automated] Fix relative figure paths in slides/ #32

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Nov 4, 2024

Automated changes by fix-figure-paths.yaml workflow.
Compare and merge to keep figure paths compatible with compilation outside of overleaf.

Explanation:

From what I understand, overleaf automatically tab-completes image paths relative to the project root, e.g.
slides/supervised-regression/figure/nutshell-regression-poly-plot-1.pdf - which is apparently fine on overleaf
but causes errors when trying to compile slides locally, where paths relative to the folder containing the
.tex file are required (i.e. figure/nutshell-regression-poly-plot-1.pdf).

As far as I understand it the latter path should also work on overleaf.
To locally auto-fix this, the following command can be used on Linux:

sed -i -E "s\slides/[0-9a-z-]+/figure\figure\g" $(find slides -iname "*.tex")

Which is exactly what this workflow does.
Note that macOS requires gnu-sed as the default BSD sed behaves differently.

@github-actions github-actions bot force-pushed the fix-figure-paths branch 9 times, most recently from caf13d7 to d8c5a59 Compare November 10, 2024 19:28
@github-actions github-actions bot force-pushed the fix-figure-paths branch 3 times, most recently from 6d993fc to d85dd7f Compare November 11, 2024 21:35
@github-actions github-actions bot force-pushed the fix-figure-paths branch 5 times, most recently from 32b23ce to a6806bd Compare November 26, 2024 13:38
@github-actions github-actions bot force-pushed the fix-figure-paths branch 6 times, most recently from 9429df8 to 4ccf08f Compare December 4, 2024 16:58
@github-actions github-actions bot force-pushed the fix-figure-paths branch 3 times, most recently from 917cf0c to f2cdc69 Compare December 10, 2024 08:15
@github-actions github-actions bot force-pushed the fix-figure-paths branch 4 times, most recently from f7ad13b to 0ad1a1d Compare January 10, 2025 21:59
@github-actions github-actions bot force-pushed the fix-figure-paths branch 7 times, most recently from c58ca8b to fde598f Compare January 14, 2025 16:21
@github-actions github-actions bot force-pushed the fix-figure-paths branch 3 times, most recently from 6c2bafc to d00b64a Compare January 20, 2025 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant