LSC
LSC
Referring to master: https://github.com/ntucllab/libact/blob/master/libact/labelers/ideal_labeler.py#L30
Current IdealLabeler seems to return a list of labels instead of the label. This should also be fixed. ``` python self.y[np.where([...])[0]] ``` should be ``` python self.y[np.where([...])[0]][0] ``` or ```...
Opened issue #32. Please solve it first.
#32 closed. Let's move discussion under #33.
See comments in #31.
Pending review. Let's leave this until 2016...
> 我覺得這個做法應該是比較正確的,不過應該不是「例如 `Repository` 應該被宣告在 `package usecase`」 而是 usecase 要去實作某個 interface 然後讓 repository 來實作他, usecase 使用這個 interface, 而這個 interface 的名字應該不是 repository,可能是按照實際需求的小 interface 這樣。 可以這樣拆分的話最好,usecase 那邊我還沒看得很仔細。 > 然後 Logger 的 interface 我看法是傾向保留,否則就變成要在各個...
其實我開了 issue 之後才看到 #21 等討論,如果已經有計畫大改的話那先放著也沒關係。 個人想法是,如果沒有替換多種實作的需求,應該就沒有必要定義 interface。