Open timonmerk opened 9 months ago
minor comment but this also includes the doc changes from #99 - I'd remove those first to keep this PR clean, thanks @timonmerk - we will review it.
Yes, sorry for that! Took me some time to figure out how to remove an existing commit from github, but git rebase and force push did the job :)
Previously I obtain an error in the test_criterions.py
but lokally I did not obtain this error. After merging the upstream main changes, the error now also doesn't occur any more.
@stes Are there any comments about the docstring and test I added?
Addresses #100 Maybe I misunderstood the function but I added keywords that the sampling can now allow for
empirical
anduniform
priors of the discrete label, and in addition to select between conditional and discrete only positive sampling.