inducer / inducer/arraycontext

Name `with_container_arithmetic` flags consistently

オープン
#279 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Python
スター
9
フォーク
11
平均マージ
17時間 12分
マージ済み PR(30日)
2

説明

> I always have a hard time remembering which options correspond to the decorated container being the outer/inner structure. Might consider (at some point, doesn't have to be here) renaming so that everything is referenced in relation to the container, e.g., `number_bcasts_across`, `bcasts_across_obj_array`, `container_types_bcast_across`?

_Originally posted by @majosm in https://github.com/inducer/arraycontext/pull/190#discussion_r1710194416_

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

まず `with_container_arithmetic` フラグの実装とすべての使用箇所を確認し、その後、命名の文脈について pull request 190 にリンクされた議論を読んでください。コンテナ相対の完全な命名スキームを決定し、影響を受けるすべての参照とそのテストを更新すれば、完了の定義になります。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
data
issue の種類
リファクタリング
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
30/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。