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

Add first Teaching LLM Assistants post #347

Merged
merged 7 commits into from
Jan 15, 2025
Merged

Conversation

tobyhodges
Copy link
Member

Tagging @ErinBecker for a review, as a listed co-author

Tagging @OscarSiba for editorial review. Feel free to edit this down as needed. I did try to keep it brief but brevity is an area for improvement for me!

Copy link

netlify bot commented Jan 7, 2025

Deploy Preview for carpentries-website ready!

Name Link
🔨 Latest commit 286c133
🔍 Latest deploy log https://app.netlify.com/sites/carpentries-website/deploys/6787fedd4273a30008b8def4
😎 Deploy Preview https://deploy-preview-347--carpentries-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@maneesha
Copy link
Contributor

maneesha commented Jan 7, 2025

I will leave feedback on content to Erin & Oscar.
I have made the following changes:

  • use relative links
  • use blog post naming conventions as described in this repo's README

@maneesha
Copy link
Contributor

maneesha commented Jan 7, 2025

Note you will not be able to preview the blog post using the Netlify preview included here as it only builds current/past posts, not future posts. I am trying to get it to build draft posts in the preview and will leave a follow up note here when I set that up.

Copy link
Contributor

@ErinBecker ErinBecker left a comment

Choose a reason for hiding this comment

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

This looks great @tobyhodges! I've added some suggestions, mostly to remove idiomatic language and to transition slashed/phrases to what I think is smoother language. None of my changes are critical. Great post!

content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved

## LLM assistants and The Carpentries Core Values
Central to the discussion in both community sessions was the question of how teaching LLMs in Carpentries workshops might align with or against [the community's Core Values](/about-us/#our-values).
Participants highlighted concerns about the lack of attribution of the data used to train the models (conflicting with our commitments to act openly and value all contributions), and the significant energy and resource requirements leading to environmental impact of the training process itself.
Copy link
Contributor

Choose a reason for hiding this comment

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

@OscarSiba - could you please provide guidance on whether our core values should be capitalized? I've most often seen them that way but am never sure.

Copy link
Contributor

Choose a reason for hiding this comment

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

On our website we chose to capitalise and italicise whenever mentioned, so I will render them that way here also. Thanks @ErinBecker.

content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
@tobyhodges
Copy link
Member Author

use blog post naming conventions as described in this repo's README

Thanks @maneesha I had forgotten about the new naming convention

various wording improvements

Co-authored-by: Erin Becker <[email protected]>
@maneesha maneesha requested a review from elletjies January 8, 2025 14:24
@maneesha
Copy link
Contributor

maneesha commented Jan 8, 2025

I've just added the ability to preview future blog posts, so you should see this rendered post if you to go the preview link for this PR: https://deploy-preview-347--carpentries-website.netlify.app/blog/

@tobyhodges
Copy link
Member Author

Amazing, thank you @maneesha.

Copy link
Contributor

@OscarSiba OscarSiba left a comment

Choose a reason for hiding this comment

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

Some changes, mainly to do with our styling conventions. Once resolved @tobyhodges, please alert @maneesha to merge.

content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved

## LLM assistants and The Carpentries Core Values
Central to the discussion in both community sessions was the question of how teaching LLMs in Carpentries workshops might align with or against [the community's Core Values](/about-us/#our-values).
Participants highlighted concerns about the lack of attribution of the data used to train the models (conflicting with our commitments to act openly and value all contributions), and the significant energy and resource requirements leading to environmental impact of the training process itself.
Copy link
Contributor

Choose a reason for hiding this comment

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

On our website we chose to capitalise and italicise whenever mentioned, so I will render them that way here also. Thanks @ErinBecker.

content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
content/posts/2025/01/teaching-llms-report.md Outdated Show resolved Hide resolved
Co-authored-by: Oscar Masinyana <[email protected]>
@tobyhodges
Copy link
Member Author

Thanks for the review, @OscarSiba. @maneesha the post is now ready to merge

@maneesha maneesha merged commit fc7e57e into carpentries:main Jan 15, 2025
4 checks passed
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.

5 participants