Skip to content

Conversation

@kartikayk
Copy link
Contributor

Context

  • As per title

Changelog

  • As per title

Test plan

  • Eyes

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 20, 2024
@netlify
Copy link

netlify bot commented Feb 20, 2024

Deploy Preview for torchtune-preview ready!

Name Link
🔨 Latest commit 54e063f
🔍 Latest deploy log https://app.netlify.com/sites/torchtune-preview/deploys/65d61f144bacf100081d5970
😎 Deploy Preview https://deploy-preview-395--torchtune-preview.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.

@msaroufim msaroufim self-requested a review February 20, 2024 17:28
Copy link
Member

@msaroufim msaroufim left a comment

Choose a reason for hiding this comment

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

Sorry what does this license mean exactly? It doesn't seem like it's quite the Llama license but something else?

@@ -0,0 +1,11 @@
Copyright 2024 Meta
Copy link
Contributor

@NicolasHug NicolasHug Feb 20, 2024

Choose a reason for hiding this comment

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

It should be more obvious when looking at the file that this is BSD-3

Suggested change
Copyright 2024 Meta
BSD 3-Clause License
Copyright 2024 Meta

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah @NicolasHug thanks so much! I actually was curious about this as well. I dug through fast gpt for example (https://github.com/pytorch-labs/gpt-fast?tab=BSD-3-Clause-1-ov-file) and didn'f see this in the raw file. I'm sure I'm missing something simple. What is it?

Copy link
Contributor

@NicolasHug NicolasHug Feb 20, 2024

Choose a reason for hiding this comment

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

GitHub can figure it out itself even if it's not explicit mentioned, but I see no reason not to include it for those who are reading this file outside of the GH UI.

All of audio, vision, text and rl make it explicit:

https://github.com/pytorch/vision/blob/main/LICENSE
https://github.com/pytorch/audio/blob/main/LICENSE
https://github.com/pytorch/text/blob/main/LICENSE
https://github.com/pytorch/rl/blob/main/LICENSE

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok that makes sense to me. Yup no reason to not make this explicit. Will update before merging.

@NicolasHug
Copy link
Contributor

@msaroufim this is BSD3, it just doesn't say it is

@kartikayk kartikayk merged commit cd7228e into main Feb 21, 2024
@joecummings joecummings deleted the license_file branch March 18, 2024 20:51
FlamingoPg pushed a commit to FlamingoPg/sgl-tune-eagle that referenced this pull request May 26, 2025
when setting `enable_memory_snapshot = true` in `.toml`
* dump memory snapshots in case of OOMs. output folder is
`memory_snapshot/iteration_x_exit`
* dump regularly according to `profile_freq`. output folder is
`memory_snapshot/iteration_x`
* existing `.toml` works since `enable_memory_snapshot=False` by default

snapshot is an example of the dump when OOM happens

<img width="1640" alt="Screenshot 2024-06-12 at 9 26 53 PM"
src="https://github.com/pytorch/torchtitan/assets/134637289/6420799c-ae68-4b35-b8bb-f5b6ab3dd053">
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants