Skip to content

Added comments about Llama3 weights to Llama tutorial - #830

Merged
ptrendx merged 18 commits into
NVIDIA:mainfrom
pggPL:Llama3-tutorial
Jun 1, 2024
Merged

Added comments about Llama3 weights to Llama tutorial#830
ptrendx merged 18 commits into
NVIDIA:mainfrom
pggPL:Llama3-tutorial

Conversation

@pggPL

@pggPLpggPL commented May 1, 2024

Copy link
Copy Markdown
Collaborator

Description

Meta released LLama 3 model in April. We have tutorial for Llama 2. It turned out that it works with Llama 3. I changed comments within tutorial. They inform that the code can be run with Llama 3 weights as well. I did not change the benchmark time(it's only Llama 2).

Type of change

  • Documentation change (change only to the documentation, either a fix or a new content)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Changes

Please list the changes introduced in this PR:

  • changes in comments and markdown part in the Llama tutorial,

Checklist:

  • I have read and followed the contributing guidelines
  • The functionality is complete
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@pggPL
pggPLforce-pushed the Llama3-tutorial branch 4 times, most recently from 4272e91 to 2a42afcCompareMay 15, 2024 18:36
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
@ptrendxptrendx added the documentation Improvements or additions to documentation label May 17, 2024
@ptrendx

Copy link
Copy Markdown
Member

@sudhakarsingh27 Please review.

pggPLand others added 12 commits May 24, 2024 15:41
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
@sudhakarsingh27

Copy link
Copy Markdown
Member

@ptrendx I've reviewed and added the necessary changes. Could you pls give it one look?

Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
@ptrendx
ptrendx merged commit 8b21049 into NVIDIA:mainJun 1, 2024
ptrendx pushed a commit that referenced this pull request Jun 1, 2024
* Llama 3 update
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
* Times update
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
* Times update
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
* utils.py fix
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
* utils.py fix
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
* utils.py fix
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
* update te llama tutorial to allow running with llama 3 weights
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
* small fixes
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
* small fix
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
* small fix
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
* add llama 3 vs llama 2 distinctions
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
* paraphrasing and corrected facts
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
* fix
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
* fix
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
---------
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Signed-off-by: Sudhakar Singh <sudhakars@nvidia.com>
Co-authored-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Co-authored-by: Sudhakar Singh <sudhakars@nvidia.com>
@pggPL
pggPL deleted the Llama3-tutorial branch August 5, 2026 16:16
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.7.0documentationImprovements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@pggPL@ptrendx@sudhakarsingh27