CpuMath Enhancement: Make bound checking of loops in hardware intrinsics more efficient
メンテナーはふだん 3 日以内に返信
まだ誰も着手していません。
- #2939 @llRandom による — マージされずにクローズ
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 78/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- csharp
- 領域
- performance
調査の方向性
まず、コンテキストを把握するために、src\Microsoft.ML.CpuMath\SseIntrinsics.cs と src\Microsoft.ML.CpuMath\AvxIntrinsics.cs のループ境界チェック、およびリンクされた issue と過去の pull request の議論を読みます。指定されたチェックを両方のファイルで一貫して更新し、その結果のループが既存の境界を維持しながら不要な命令を削除していることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Style changes needed to solve part of https://github.com/dotnet/machinelearning/issues/823
Details
- In
src\Microsoft.ML.CpuMath\SseIntrinsics.csandsrc\Microsoft.ML.CpuMath\AvxIntrinsics.cs, changingwhile (pDstCurrent + 4 <= pDstEnd)for the loop bound checking intowhile (pDstCurrent <= pDstEnd - 4)to save an instruction (ref: https://github.com/dotnet/machinelearning/pull/668#issuecomment-412121893) - It may probably be a CoreCLR issue
- 主要言語
- C#
- スター
- 9.4k
- フォーク
- 2k
- 平均マージ
- 3日 3時間
- マージ済み PR（30日）
- 15
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
dotnet/machinelearning のほかの issue
-
LlamaTokenizer encodes some runs of repeated characters differently from SentencePiece対応中かも @Laurianti が 12 日前に担当しました。 オープンuntriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
dotnet/machinelearning#7751 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
BertTokenizer merges words separated only by `\n`, `\t` or `\r`対応中かも @rosebyte が 20 日前に担当しました。 オープンarea-Tokenizers green
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
dotnet/machinelearning#7724 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 3 日以内に返信
-
area-TimeSeries documentation untriaged
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
dotnet/machinelearning#7205 ·
メンテナーはふだん 3 日以内に返信
-
area-ONNX documentation Priority:3
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
dotnet/machinelearning#5894 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
Add a .vsconfig fileオープンarea-Infrastructure Priority:2
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
dotnet/machinelearning#3260 · コメント 2 件 · リアクション 1 件 ·
メンテナーはふだん 3 日以内に返信
dotnet/machinelearning の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
area:re1 bounty help wanted needs-game
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
StackExchange/StackExchange.Redis#3274 ·
メンテナーはふだん 1 日以内に返信
-
affected/v17 affected/v18 area/frontend type/bug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
umbraco/Umbraco-CMS#24186 ·
メンテナーはふだん 2 日以内に返信