Skip to content

remove the undefined variable in ckpt testing #67

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.

Sign up for GitHub

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

Jump to bottom
Merged
eltonzheng merged 1 commit into from
Feb 12, 2020
Merged

remove the undefined variable in ckpt testing #67

eltonzheng merged 1 commit into from
Feb 12, 2020

Conversation

@eltonzheng
Copy link
Contributor

@eltonzheng eltonzheng commented Feb 12, 2020

No description provided.

@eltonzheng eltonzheng requested review from a team, ShadenSmith and jeffra February 12, 2020 19:22
@eltonzheng eltonzheng linked an issue Feb 12, 2020 that may be closed by this pull request
Undefined name: f --> bare except #66
Closed
@eltonzheng eltonzheng merged commit 4f7d016 into master Feb 12, 2020
@eltonzheng eltonzheng deleted the fix_ckpt_bug branch February 12, 2020 20:21
kouml pushed a commit to kouml/DeepSpeed that referenced this pull request Apr 3, 2020
@eltonzheng
remove the undefined variable in ckpt testing (deepspeedai#67)
cf102d9
samyam added a commit that referenced this pull request Nov 24, 2020
@jeffra @samyam
Build tensors on gpu by default (#67)
74bd28f 
* build new tensors on cuda device and not cpu (reduces model build time)

* import os

Co-authored-by: Samyam Rajbhandari <samyamr@microsoft.com>
samyam added a commit that referenced this pull request Nov 24, 2020
@jeffra @samyam @tjruwase
Build tensors on gpu by default (#67)
08e6f5a 
* build new tensors on cuda device and not cpu (reduces model build time)

* import os

Co-authored-by: Samyam Rajbhandari <samyamr@microsoft.com>
delock pushed a commit to delock/DeepSpeedSYCLSupport that referenced this pull request Sep 21, 2022
@guoyejun
update submodule deepspeedexamples to first check if sysmon is there (d…
188ac54 
…eepspeedai#67)

commit 78c17c85fdb484f7184d8e81555855c08f5f89fe (HEAD, origin/xpu-main, origin/HEAD)
Author: Guo Yejun <yejun.guo@intel.com>
Date:   Thu Sep 1 05:48:45 2022 -0700

    pretrain_gpt2.py: first check if sysmon is there
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tjruwase tjruwase tjruwase approved these changes

@ShadenSmith ShadenSmith Awaiting requested review from ShadenSmith

@jeffra jeffra Awaiting requested review from jeffra

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Undefined name: f --> bare except

3 participants

@eltonzheng @tjruwase