Pymc3: Add asymetric laplace distribution

[twiecki]

There is an implementation here we could port quite easily: https://github.com/epidemics/COVIDNPIs/blob/manuscript/epimodel/pymc3_distributions/asymmetric_laplace.py

[chandan5362]

Hey @twiecki , could you please explain the issue a bit more. I would love to work on this issue.

[twiecki]

@chandan5362 That's great. As you can see in the link, the distribution is already implemented. So you would copy & paste that into distributions/continuous.py, make sure the doc string matches the ones we already have (e.g. creates a plot) and add unittests.

[chandan5362]

@twiecki Thanks for the help. I am little confused here. where should I add unittests ?. whether should I create separate file or I will have to add it to the existing one.

[ricardoV94]

@chandan5362 you should add your unit test in the existing files with unit tests for the other distributions:
https://www.github.com/pymc-devs/pymc3/tree/master/pymc3%2Ftests%2Ftest_distributions.py

You can also have a look and check what kind of tests are there for the existing distributions.

[CloudChaoszero]

If there are no assignees to this sometime soon, I am happy to submit a PR in the near future.

• Feel free to @ me.

[twiecki]

@chandan5362 Are you actively working on this?
@CloudChaoszero I would say you can start tomorrow if you we don't hear.

[chandan5362]

@twiecki @CloudChaoszero yeah, I am still working on it and I will make a PR tomorrow or day after tomorrow.

[CloudChaoszero]

@twiecki @CloudChaoszero yeah, I am still working on it and I will make a PR tomorrow or day after tomorrow.

Oh hey, no worries @chandan5362, just checking to see if it was officially active. haha
(Take your time, it's the holidays 😄)