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

Bump composer version #84

Merged
merged 14 commits into from
May 9, 2023
Merged

Bump composer version #84

merged 14 commits into from
May 9, 2023

Conversation

vchiley
Copy link
Contributor

@vchiley vchiley commented May 9, 2023

Was asked to run try this streaming branch out, so I tested updating composer as well. No issues; seems slightly faster on 125m model (might just be runtime variance, but I'm not sure)
Screenshot 2023-05-09 at 12 57 20 PM
Screenshot 2023-05-09 at 12 57 53 PM

@vchiley vchiley self-assigned this May 9, 2023
Copy link
Collaborator

@mvpatel2000 mvpatel2000 left a comment

Choose a reason for hiding this comment

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

Raising min should not change anything (it will always use 14.1). but we should raise pin anyways

@vchiley vchiley merged commit 6035041 into mosaicml:main May 9, 2023
6 checks passed
bmosaicml pushed a commit that referenced this pull request Jun 6, 2023
* Adding Nemo Megatron readme and example yaml files

* making assets

* Update README.md

Updating nemo readme with assets

* Update README.md

Updating README wording

* removing stray assets

* Moving around files

* Cleaning up abbreviations, scripts

* Update nemo/README.md

Co-authored-by: Daniel King <43149077+dakinggg@users.noreply.github.com>

* Moving to under examples repo

---------

Co-authored-by: Brandon Cui <bcui@Brandons-MacBook-Pro.local>
Co-authored-by: Daniel King <43149077+dakinggg@users.noreply.github.com>
bmosaicml pushed a commit that referenced this pull request Jun 6, 2023
@vchiley vchiley deleted the updt_cmpsr branch November 9, 2023 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants