Rindrics / lucifer

ネ申エクセルと戦うためのRパッケージ
GNU General Public License v3.0
3 stars 0 forks source link

extract_clusters() を一般化する #68

Closed Rindrics closed 5 years ago

Rindrics commented 5 years ago
image

クラスタが複数なくても,図のようにデータがセルA1から始まっていないときに結局 extract_clusters() を発動することになった.

この関数はもっと一般化して,detect_topleft() のような名前にする必要がある

Rindrics commented 5 years ago

いや,ends も設定しているから topleft ではだめか.

append_info() で付加情報も追加しているので,結局 make_data() していることになっている. 関数の機能がもっとシンプルになるように整理が必要か?

Rindrics commented 5 years ago

少なくともdir = row や dir = col は反復のないデータを扱う場合、引数名がそぐわない(けどオプションとしてどちらかを選ばなければならない 反復がない場合、最初のヘッダ行特定のための検索は、列方向に制限して引数を一つ減らす

rep_dir = とすべき

Rindrics commented 5 years ago

resolved by https://github.com/kikirinrin/lucifer/commit/f4ff5c2c9423fdaaf6f2abb040370c693fd56663