Tim Mensinger
Tim Mensinger
In commit (d95a7cfa0cc019b10bd0738fbd186f4e73c81613) I implemented a wrapper class `CausalForest` which implements the methods `fit`, `predict`, `save` and `load`. What remains to be done: 1. Check if I can hide private...
1. Make sure that all docstrings in code are correctly formatted 2. Include example code in rtd?
- [x] Create an example notebook for rtd (e21deac7) - [ ] Polish details in documentation
Hello spot92, thanks for finding my repository and playing around with my code. I presume you were running the code on a Windows machine. Unfortunately I haven't tested the code...
Hey @yangyijane, thanks for reaching out! Could you give me some more information? Did you use an environment file, or how did you create the environment? Preferably you provide a...
I added unit tests for functions 1. `_compute_valid_splitting_indices` 2. `_retrieve_index` 3. `_predict_row_causaltree`
For now it would be especially important to add unit tests for functions located in ``cforest/forest.py``.
I renamed the python package to `cforest`. **Next step:** Rename github repo from `causal-forest` to `cforest`?
**What has been done:** The commits (aca31b1ea3f76964) and (a2f504f9ccfbab8cd9) improve the speed of the inner loop (over observations) by a big margin. In the first commit I changed most `np.sum()`...
Hello! Thanks for opening an issue and for digging so deep yourself. This is indeed a bug, and we need to fix it. We will try to fix it with...