Aleksander Molak comments

Results 23 comments of


                                            Aleksander Molak

Allow more algorithm to use prior knowledge

@shaido987, is there a way to help with this task as a contributor?

Does `cosinesimil` return similarity or distance?

Thank you for a quick reply @searchivarius! Do you think it'd be useful to update the naming and/or clarify this in the docs? If so, I am happy to help...

Does `cosinesimil` return similarity or distance?

Hi @searchivarius, I am sorry for the delayed reply. I am happy to clarify the docs and add a synonym / alias. I could start working on this next Tuesday....

Does `cosinesimil` return similarity or distance?

Great, @searchivarius, I'll fork it and work on it later today.

UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 2810: character maps to <undefined>

I experience the same issue. Pandas reads the file with defualt settings (`utf-8`), but `CSVLoader` fails wih `UnicodeDecodeError`

UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 2810: character maps to <undefined>

Hi @levalencia I changed the encoding to `8859` and it worked. That said, I am still not sure what caused the error as `pandas` opens the same file without issues....

Targeted Maximum Likelihood Estimator (TMLE) in EconML

Thanks for the reply @kbattocchi Is there a particular reason behind your suggestion regarding the binary outcome? Do you think that adding continuous outcomes would not be useful in the...

Can I add restrictions on causal structure？

You can now (1.0.3) add causal knowledge to the PC algorithm in gCastle, @xyb151158

Allow more algorithm to use prior knowledge

@shaido987 It seems that there were some attempts to introduce the possibility for adding causal knowledge to `DirectLiNGAM` (https://github.com/huawei-noah/trustworthyAI/blob/master/gcastle/castle/algorithms/lingam/direct_lingam.py), but it seems it does not work in the current version...

`distance_matching` estimator raises an exception when binary treatment encoded using `int`s

@amit-sharma, I see. How about adding a simple check if the unique values are `0` and `1`? It seems to be a very common way to encode binary treatments in...