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

Adding NeMo Curator #382

Closed
wants to merge 0 commits into from
Closed

Adding NeMo Curator #382

wants to merge 0 commits into from

Conversation

exactlyallan
Copy link
Member

Adding NeMo curator section per #376 request. Thanks for your patience.
Some other minor content fixes.

Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

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

This PR looks like its git commit history is not quite right. Can you rebase these changes on the latest main branch, or otherwise fix the diffs that are showing up from previously-merged PRs?

cursor: pointer;
text-decoration: none;
Copy link
Contributor

Choose a reason for hiding this comment

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

This is defining text-decoration twice.

Suggested change
text-decoration: none;

go.sum Outdated
@@ -1,4 +1,8 @@
github.com/FortAwesome/Font-Awesome v0.0.0-20230327165841-0698449d50f2/go.mod h1:IUgezN/MFpCDIlFezw3L8j83oeiIuYoj28Miwr/KUYo=
github.com/FortAwesome/Font-Awesome v0.0.0-20240402185447-c0f460dca7f7/go.mod h1:IUgezN/MFpCDIlFezw3L8j83oeiIuYoj28Miwr/KUYo=
Copy link
Contributor

Choose a reason for hiding this comment

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

Why does this have checksums for multiple versions? If we updated a dependency, we do not need to keep the old checksums for versions we no longer depend on. Is this automatically generated? Maybe delete it and regenerate it?

@@ -1,12 +1,12 @@
{
"name": "rapids.ai",
"name": "rapids.ai_ae",
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did this change?

Copy link
Member Author

Choose a reason for hiding this comment

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

I have multiple copies of RAPIDS repo from different forks.

@exactlyallan
Copy link
Member Author

This PR looks like its git commit history is not quite right. Can you rebase these changes on the latest main branch, or otherwise fix the diffs that are showing up from previously-merged PRs?

Yeah not sure what happened - I think Ill rebase and try again next change I have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants