dowhy icon indicating copy to clipboard operation
dowhy copied to clipboard

Eli a look at synthetic data generation - linear_dataset

Open elikling opened this issue 3 years ago • 2 comments

Signed-off-by: Eli Kling [email protected]

started an example notebook showcasing the functions for generating synthetic data.

for now just did linear_dataset. This is a good time to get feedback before moving on to the next functions.

I did run into some issues. For one of them I opend an Issue #664. The others could be due to some problem my side (not all function parms recognised)

I realise there is a question of whether this should be part of the documentation. My view is that there is too much code for it to be in the documentations and this is a good way to allow playing with the function and undersdanding it.

elikling avatar Sep 29 '22 21:09 elikling

How come this did not pass the DCO check? I ensured the commit had the required text!

elikling avatar Sep 29 '22 21:09 elikling

The issues I mentioned in the notes were resolved when I refreshed doWhy :-) - pull request is relvant - I will correct my notes in the next round.

elikling avatar Sep 30 '22 15:09 elikling

thanks @elikling for adding this. Based on our discussion in the last meeting, do you think this information will be better as a docstring for the linear_dataset function? I'd encourage you to add the parameter description and the code example as a docstring to the function here, so that users can find this information easily.

amit-sharma avatar Nov 10 '22 12:11 amit-sharma

Yep - it makes sense to update to Docstrings,

I read up alittle on Docstrings and did my best - see updated datasets.py

Eli

elikling avatar Nov 25 '22 16:11 elikling

I am closing this pull request as it is now messy - will follow advice and create a fresh branch and a new pull request :-)

elikling avatar Nov 29 '22 13:11 elikling