Skip to content

Conversation

@j-atkins
Copy link
Collaborator

Add presentation material (Quarto slides source code and rendered HTML) plus performance testing/plotting scripts to teacher-content in docs.

@j-atkins j-atkins requested a review from ammedd November 24, 2025 09:46
@j-atkins
Copy link
Collaborator Author

j-atkins commented Nov 24, 2025

I realise it's not ideal uploading all the Presentation_files (not least because it is probably the cause of failures on the pre-commit checks)... will need to have a think on how to add presentation material without this and ideally keeping all figures etc. rendering properly!

@ammedd
Copy link
Collaborator

ammedd commented Nov 24, 2025

I like that you're adding this, but it's not teacher content in my opinion. More for us as developers/promotors

@ammedd
Copy link
Collaborator

ammedd commented Nov 24, 2025

I realise it's not ideal uploading all the Presentation_files (not least because it is probably the cause of failures on the pre-commit checks)... will need to have a think on how to add presentation material without this and ideally keeping all figures etc. rendering properly!

Indeed. I'd try to keep the .qmd file on GitHub but not the rest of the Presentation_files. Would be nice to include an html page or link to quarto-pub in the docs though...

@j-atkins
Copy link
Collaborator Author

Thanks @ammedd, I've moved these presentation materials to a new developer-content directory now 👍

Also, the presentation .qmd file is now rendered as self-contained and so avoids generating the Presentation_files. The HTML should be viewable when opening the file in a browser!

Note, I've adjusted the pre-commit config to exclude the presentation files from the prettier hook - this was being overwhelmed previously by the size of the files.

@j-atkins j-atkins changed the title Add Parcels meeting presentation material to teacher-content Add Parcels meeting presentation material to developer-content Nov 24, 2025
rev: v3.5.1
hooks:
- id: prettier
exclude: "docs/user-guide/developer-content/parcels-meeting-Nov2025/presentation/.*" # these quarto files break prettier
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make more generic?

@j-atkins j-atkins merged commit 3a53b52 into main Nov 24, 2025
10 checks passed
@j-atkins j-atkins deleted the add-parcels-presentation branch November 24, 2025 14:48
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.

3 participants