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.

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

Fix cifar tutorial links #68

Closed
jeffra opened this issue Feb 12, 2020 · 1 comment · Fixed by #69
Closed

Fix cifar tutorial links #68

jeffra opened this issue Feb 12, 2020 · 1 comment · Fixed by #69
Labels
bug Something isn't working documentation Improvements or additions to documentation good first issue Good for newcomers

Comments

@jeffra
Copy link
Contributor

jeffra commented Feb 12, 2020

There are two dead links at top here: https://github.com/microsoft/DeepSpeed/blob/master/docs/tutorials/CIFAR-10.md
@jeffra jeffra added the documentation Improvements or additions to documentation label Feb 12, 2020
@ShadenSmith ShadenSmith added bug Something isn't working good first issue Good for newcomers labels Feb 12, 2020
@ShadenSmith
Copy link
Contributor

I believe those two links should be combined and simply link to the getting started guide: ../../README.md#getting-started

@ShadenSmith ShadenSmith linked a pull request Feb 12, 2020 that will close this issue
Fix broken links in CIFAR #69
Merged
jeffra pushed a commit to jeffra/DeepSpeed that referenced this issue Aug 28, 2020
@arashashari @jeffra
Adding/updating sparsity config (microsoft#68)
f1b3472 
* adding/updating sparsity config patterns

* adding random to Variable sparsity

* fixing a typo

* applying comment adding missing argument docstring
jeffra pushed a commit to jeffra/DeepSpeed that referenced this issue Aug 28, 2020
@arashashari @jeffra
adding unit test/s for sparse transformer (microsoft#60)
b6435ba 
* adding unit test/s for sparse transformer

* file-name change update

* updated tests based on new list of sparsity configs

* Adding/updating sparsity config (microsoft#68)

* adding/updating sparsity config patterns

* adding random to Variable sparsity

* fixing a typo

* applying comment adding missing argument docstring

* adding unit test/s for sparse transformer

* file-name change update

* updated tests based on new list of sparsity configs

* skipping a test if it is run on gpu with compute capability < 7; minimum V100
jeffra pushed a commit to jeffra/DeepSpeed that referenced this issue Aug 28, 2020
@arashashari @jeffra
updating deepspeed config for Sparse Transformer (microsoft#62)
dce7755 
* updating deepspeed config for Sparse Transformer

* Adding/updating sparsity config (microsoft#68)

* adding/updating sparsity config patterns

* adding random to Variable sparsity

* fixing a typo

* applying comment adding missing argument docstring

* updating deepspeed config for Sparse Transformer

* updating sparsity config for DeepSpeed parameter list

* adding unit test/s for sparse transformer (microsoft#60)

* adding unit test/s for sparse transformer

* file-name change update

* updated tests based on new list of sparsity configs

* Adding/updating sparsity config (microsoft#68)

* adding/updating sparsity config patterns

* adding random to Variable sparsity

* fixing a typo

* applying comment adding missing argument docstring

* adding unit test/s for sparse transformer

* file-name change update

* updated tests based on new list of sparsity configs

* skipping a test if it is run on gpu with compute capability < 7; minimum V100

* fix a naming issue in utils file: bert_mode -> bert (microsoft#69)

* updating deepspeed config for Sparse Transformer

* updating sparsity config for DeepSpeed parameter list
delock pushed a commit to delock/DeepSpeedSYCLSupport that referenced this issue Sep 21, 2022
@guoyejun
update submodule deepspeedexamples to add xpu sync after train_step (m…
759a168 
…icrosoft#68)

commit 80d1d4a13fe5e2a20a276d62a28cb69da782ffa7 (HEAD, origin/xpu-main, origin/HEAD)
Author: Guo Yejun <yejun.guo@intel.com>
Date:   Sun Sep 4 02:12:29 2022 -0700

    pretrain_gpt2.py: add xpu sync for safty after train_step
pengwa pushed a commit to pengwa/DeepSpeed that referenced this issue Oct 14, 2022
@rraminen
Enable Megatron-LM workload on ROCm (microsoft#68)
b4d4a0e 
* Enable Megatron-LM workload on ROCm (microsoft#1)

* Enable Megatron workload on ROCm

* Added ds_pretrain_gpt_350M_dense_pipeclean.sh

* removed a file

* Removed an extra line

* Fix to resolve the below rsqrtf() error on ROCm

/root/Megatron-DeepSpeed/megatron/fused_kernels/layer_norm_hip_kernel.hip:298:10: error: no matching function for call to 'rsqrtf'
  return rsqrtf(v);
         ^~~~~~
/opt/rocm-5.2.0/llvm/lib/clang/14.0.0/include/__clang_hip_math.h:521:7: note: candidate function not viable: call to __device__ function from __host__ function
float rsqrtf(float __x) { return __ocml_rsqrt_f32(__x); }
      ^

* Simplified code

* Simplified the code

* Removed extra spaces
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working documentation Improvements or additions to documentation good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix broken links in CIFAR microsoft/DeepSpeed
2 participants
@ShadenSmith @jeffra