google-deepmind / torch-distributions

Probability distributions, wrapped for Torch.
BSD 3-Clause "New" or "Revised" License
61 stars 25 forks source link

Some randkit generators missing docs? #11

Closed jucor closed 10 years ago

jucor commented 10 years ago

Issue by d11 from Monday Nov 11, 2013 at 10:53 GMT Originally opened as https://github.com/jucor/torch-randomkit/issues/24


e.g. long; ulong. http://jucor.github.io/torch-randomkit/randomkit.html#long

They appear in the menu but have no doc section.

jucor commented 10 years ago

Comment by jucor from Monday Nov 11, 2013 at 10:54 GMT


Nice. I noticed some of those already and removed them, but apparently there's some leftover. That comes from the automatically generated HTML page, which has the same problem.

jucor commented 10 years ago

Comment by d11 from Monday Nov 11, 2013 at 10:57 GMT


At least in the case of u?long, it's because those functions aren't exposed in numpy. I guess we should just add docs manually for those, since we went to the trouble of wrapping and writing tests for them.

On 11 Nov 2013, at 10:54, Julien Cornebise notifications@github.com wrote:

Nice. I noticed some of those already and removed them, but apparently there's some leftover. That comes from the automatically generated HTML page, which has the same problem.

— Reply to this email directly or view it on GitHub.

jucor commented 10 years ago

Comment by d11 from Monday Nov 11, 2013 at 11:00 GMT


Ditto 'gauss'.

On 11 Nov 2013, at 10:57, Daniel Horgan daniel.horgan@deepmind.com wrote:

At least in the case of u?long, it's because those functions aren't exposed in numpy. I guess we should just add docs manually for those, since we went to the trouble of wrapping and writing tests for them.

On 11 Nov 2013, at 10:54, Julien Cornebise notifications@github.com wrote:

Nice. I noticed some of those already and removed them, but apparently there's some leftover. That comes from the automatically generated HTML page, which has the same problem.

— Reply to this email directly or view it on GitHub.

jucor commented 10 years ago

Comment by d11 from Monday Nov 11, 2013 at 11:13 GMT


full list is

long ulong gauss double interval noncentral_chisquare

jucor commented 10 years ago

Not relevant to this repository (moved here as part of automatic move). Closing.