Daniele Nerini
Daniele Nerini
sorry for taking so long, I think we can merge this bugfix if @feldmann-m has no objections to this?
simple code to resample the distributions: ``` def resample_distributions(a, b, weight): '''merge two distributions''' assert a.size == b.size asort = np.sort(a.flatten())[::-1] bsort = np.sort(b.flatten())[::-1] n = asort.shape[0] # resample idxsamples...
> I agree with @aitaten that this would fit well in the post-processing and that we then call the method just like we call "cdf" now as prob_matching method. Not...
closing this for now, see also #339.
Hi @SuryanarayanaY, just checking in, are you still working on this issue? Tagging @mehtamansi29 and @sachinprasadhs for input on the next steps, does this needs reassignment? Thanks!
should this be added to the hackathon?
> I think this can soon be merged, although it shows that the current way the plugins are structured is hitting its limits. It leads to a lot of duplicate...
Yes please!
short comment: when there are so many variables given and returned by functions, it's usually a good indication that you need a class. so why not refactoring the code into...
sounds very good!