Collectorのコンストラクタの引数の絶対パスの真偽値

[kyaoNK]

たぶん，この第3引数絶対パスがあるのはdatabase(Directory)に渡す際に，絶対パスが必要だからだと思うけど，ユーザ視点からすると，入れているのが絶対パスかどうかは見たら分かる．引数に必要な情報ではない気がするから，絶対パスか相対パスかは正規表現使ったり，判定するようにして内部処理で完結させたほうが良い気がする

[MTamon]

その絶対パスの真偽値はそういう使い方ではない。
収集した後のパスを、絶対パスで返すかどうかという引数。

[MTamon]

Directoryは指定したパスを基準に、ファイルのパスを生成するので、絶対パスを渡すと、絶対パスで帰ってくる。
あとややこしいのは、abspath=Falseでも相対パスが返ってくるかわからないという点。
ユーザがCollectorに絶対パスを渡せば、絶対パスで帰ってくる

[kyaoNK]

絶対パスに統一してしまったら？

[MTamon]

でもなー、ユーザが相対パスで入力して、絶対パスで返してしまうと、
パスの変更・追記箇所が分かりづらくなる問題があるんよな。
つまり、ユーザはデータベースのルートディレクトリを指定して、それが相対パスで、結果も相対パスで返ってくれば、変更後のパスの前方とユーザが渡したパスは一致する。けど、絶対パスで返すとそうはならないやん？

[kyaoNK]

理解した

[MTamon]

@kyaoNK に協力を依頼したのはこういった、直したいけど、直したら可用性が下がる問題の解決策が欲しいからでもあるから、気にせず、どんどん意見だして欲しい

[kyaoNK]

今やっとdockerで環境作れたから今から動かしてみる

[MTamon]

使い方は、僕に聞くか、充実していない＆古いReadmeか、SLP_LAB/HME/HME_DM/post_process.py を見てくれると使用例が載ってる

[MTamon]

SLP_LABは別リポジトリね

[kyaoNK]

READMEのコード動かしたけど、ファイル名がないんだけど，どういうこと？

[MTamon]

READMEを現在の仕様に合わせて更新した

[MTamon]

data_collector.pyを実行してもらった方が分かりやすいかも
プログラム中に登場するパスは、適宜自分の環境に合わせて変えてみてね

[MTamon]

@kyaoNK あと、その他のバグも発見したから、それも修正した

[kyaoNK]

実行できた

[kyaoNK]

Conditionの中身見れるようにしたいんだけど，実装してる？
ないならissue立てよう