-
Notifications
You must be signed in to change notification settings - Fork 91
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
Blog post: LLMs Don't Hallucinate #233
Blog post: LLMs Don't Hallucinate #233
Conversation
d348a4c
to
d00a9ad
Compare
d00a9ad
to
bb2406e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a really fun and interesting read, just have some points mainly about grammar!
Co-authored-by: lhancock-scottlogic <[email protected]>
"calling this narrative in this question" -> "calling this narrative into question" Co-authored-by: lhancock-scottlogic <[email protected]>
Co-authored-by: lhancock-scottlogic <[email protected]>
Co-authored-by: lhancock-scottlogic <[email protected]>
The figures are purely decorative, so role=presentation is appropriate, but pa11y pointed out that assistive technology may still treat child elements of the figure as non-decorative, which is not what I intended. So I've marked all the children of the figures as role=presentation too.
5eadd87
to
888486b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jcarstairs-scottlogic I'm fairly sure this file shouldn't have been merged. There are now two blog posts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I didn't spot that - @jcarstairs-scottlogic am I ok to delete the earlier date?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NVM, I see the other PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mgriffin-scottlogic I've made a PR to remove the earlier post
Please add a direct link to your post here:
https://jcarstairs-scottlogic.github.io/blog/2024/09/10/llms-dont-hallucinate.html
Have you (please tick each box to show completion):
npm install
followed bynpx mdspell "**/{FILE_NAME}.md" --en-gb -a -n -x -t
if that's your thing)Posts are reviewed / approved by your Regional Tech Lead.