Unused variable in OOB and UOB #28
Description
Hello,
First of all I would like to congratulate you for developing this very nice package!
I have a question regarding the files strlearn/ensembles/OOB.py and strlearn/ensembles/UOB.py.
If we look at line 28 in the OOB file and line 29 in the UOB file, we observe that the variable self.current_ctdcs_ is declared. I wonder whether there is a typo in the name of that variable because it is never used. If I am not mistaken, the variable self.current_ctdcs_ should be replaced with self.current_tdcs_ because the former corresponds to the latter with an extra (unintended) character. This way, when considering the first instance in a new chunk (different from the first one), we can reuse the class sizes defined for the last instance in the previous chunk (for this we would need self.current_tdcs = self.last_instance_sizes), which would enable us to be consistent with equation (1) from ref. [1] (i.e., research article in which OOB and UOB were introduced) at all time points.
Do you (@xehivs) also agree that this is a typo?
Many thanks for your attention!
References:
[1] S. Wang, L. L. Minku, and X. Yao, “Resampling-Based Ensemble Methods for Online Class Imbalance Learning,” IEEE Transactions on Knowledge and Data Engineering, vol. 27, no. 5, pp. 1356–1368, May 2015, doi: 10.1109/TKDE.2014.2345380.