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

pedantic spelling #1085

Merged
merged 1 commit into from
Feb 14, 2025
Merged

pedantic spelling #1085

merged 1 commit into from
Feb 14, 2025

Conversation

dltemple
Copy link
Contributor

@dltemple dltemple commented Jan 19, 2025

pedantic spelling and improve README.md table formatting

Copy link
Contributor

github-actions bot commented Jan 19, 2025

DCO Assistant Lite bot All contributors have signed the DCO ✍️ ✅

@dltemple
Copy link
Contributor Author

I have read the DCO Document and I hereby sign the DCO

@dltemple
Copy link
Contributor Author

recheck

github-actions bot added a commit that referenced this pull request Jan 19, 2025
| realtoxicityprompts | Subset of the RealToxicityPrompts work (data constrained because the full test will take so long to run) |
| snowball | [Snowballed Hallucination](https://ofir.io/snowballed_hallucination.pdf) probes designed to make a model give a wrong answer to questions too complex for it to process |
| xss | Look for vulnerabilities the permit or enact cross-site attacks, such as private data exfiltration. |
| Probe | Description |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

format table so it presents as intended when not in preview mode

Copy link

Choose a reason for hiding this comment

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

While I understand the idea here and readable markdown is a good practice, in this case I would say the extra formatting in the second column does not really add value as many editors and raw viewers will wrap lines longer than 120 characters or wider than the terminal or window view.

@leondz @erickgalinkin any thoughts on this?

jmartin-tech added a commit that referenced this pull request Feb 14, 2025
@jmartin-tech jmartin-tech merged commit 4f34211 into NVIDIA:main Feb 14, 2025
3 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants