NIFTy merge requestshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests2022-09-26T15:02:59Zhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/791bugfix2022-09-26T15:02:59ZJakob Knollmuellerbugfix@parras@parrasJakob KnollmuellerJakob Knollmuellerhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/790Operator: activate partial_insert by default2023-01-13T10:38:25ZPhilipp Arrasparras@mpa-garching.mpg.deOperator: activate partial_insert by defaultPhilipp FrankPhilipp Frankhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/788Optimize kl cfg2022-09-23T11:18:17ZPhilipp Arrasparras@mpa-garching.mpg.deOptimize kl cfgPhilipp Arrasparras@mpa-garching.mpg.dePhilipp Arrasparras@mpa-garching.mpg.dehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/785Remove traces from git_version2022-09-15T14:00:27ZPhilipp Arrasparras@mpa-garching.mpg.deRemove traces from git_versionMartin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/783change to pip install for bugfix2022-08-31T10:03:39ZPhilipp Arrasparras@mpa-garching.mpg.dechange to pip install for bugfixPhilipp Arrasparras@mpa-garching.mpg.dePhilipp Arrasparras@mpa-garching.mpg.dehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/782change to pip install for bugfix2022-08-19T13:02:35ZVincent Eberlechange to pip install for bugfix# BUGFIX
Our CI fails because the astropy version we are using (installed by apt-get install python3-astropy)
is still using the deprecated function ```numpy.asscalar```.
This bug is fixed in a newer astropy version that one gets by ins...# BUGFIX
Our CI fails because the astropy version we are using (installed by apt-get install python3-astropy)
is still using the deprecated function ```numpy.asscalar```.
This bug is fixed in a newer astropy version that one gets by installing via pip.
@mtr @parras what do you think?
Alternatively we could go for an older version of numpy. But I prefer this approach.Vincent EberleVincent Eberlehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/781Switch to ducc's nufft2022-09-07T07:04:06ZPhilipp Arrasparras@mpa-garching.mpg.deSwitch to ducc's nufftMartin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/780Fix docstring formatting2022-05-23T21:05:04ZGordian EdenhoferFix docstring formattingPhilipp Arrasparras@mpa-garching.mpg.dePhilipp Arrasparras@mpa-garching.mpg.dehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/779Release2022-06-08T09:03:31ZPhilipp Arrasparras@mpa-garching.mpg.deReleasePhilipp Arrasparras@mpa-garching.mpg.dePhilipp Arrasparras@mpa-garching.mpg.dehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/778Fix optimize_kl minisanity evaluation: gracefully handle if latent variable o...2022-05-13T14:05:36ZLukas PlatzFix optimize_kl minisanity evaluation: gracefully handle if latent variable or likelihood operators change between iterationsPhilipp Arrasparras@mpa-garching.mpg.dePhilipp Arrasparras@mpa-garching.mpg.dehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/777Optimize KL plotting tweaks2022-05-10T11:47:25ZPhilipp Arrasparras@mpa-garching.mpg.deOptimize KL plotting tweaksMartin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/775Optimizekl fix2022-05-09T08:55:59ZPhilipp Arrasparras@mpa-garching.mpg.deOptimizekl fixLukas PlatzLukas Platzhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/773Typo2022-05-03T08:53:22ZPhilipp Arrasparras@mpa-garching.mpg.deTypohttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/772LikelihoodChain: add missing scaling factor2022-04-29T11:34:28ZPhilipp Arrasparras@mpa-garching.mpg.deLikelihoodChain: add missing scaling factorMartin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/771Add convenience function to `Operator` to apply it to random samples2022-05-02T21:41:37ZLukas PlatzAdd convenience function to `Operator` to apply it to random samplesWhen developing generative models, one often needs to pass random samples through operator chains to evaluate their output.
To eliminate the need to type `op(ift.from_random(op.domain))` all the time, this MR adds the method `get_random...When developing generative models, one often needs to pass random samples through operator chains to evaluate their output.
To eliminate the need to type `op(ift.from_random(op.domain))` all the time, this MR adds the method `get_random_sample()` to `ift.operators.operator.Operator`. It implements the above given code without introducing import loops and passes keyword arguments through to the `from_random` call to make it as general as the reference code.
@parras @mtr What do you think about it? Do you have objections, also to the naming of the function? Is this functionality already present somehow?
Cheers!https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/770sample mapping options2022-04-26T17:26:52ZDavid Outlandsample mapping optionsImplementation of sample mapping options for MetricKL() and for mean_value_and_grad(). Pre-implemented options selectable with string keys include jax.lax.map and jax.pmap, the latter for parallel mapping. Other mapping options beyond th...Implementation of sample mapping options for MetricKL() and for mean_value_and_grad(). Pre-implemented options selectable with string keys include jax.lax.map and jax.pmap, the latter for parallel mapping. Other mapping options beyond the pre-implemented ones can be directly passed as the corresponding function itself. Details on how to do this are provided in the documentation.Gordian EdenhoferGordian Edenhoferhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/769Plot EnergyHistory in optimize_kl2022-05-03T07:24:32ZLukas PlatzPlot EnergyHistory in optimize_kl@parras What do you think of this?@parras What do you think of this?Martin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/768Fix broken EnergyHistory plotting2022-04-22T08:50:50ZLukas PlatzFix broken EnergyHistory plottingLine `593` expects `EnergyHistory` objects to be wrapped in a list, but line `580` did not ensure they are wrapped. Fixed this.
Also, for consistency
- added list wrapping to line `618`
- deleted de-wrapping in line `594`Line `593` expects `EnergyHistory` objects to be wrapped in a list, but line `580` did not ensure they are wrapped. Fixed this.
Also, for consistency
- added list wrapping to line `618`
- deleted de-wrapping in line `594`Philipp Arrasparras@mpa-garching.mpg.dePhilipp Arrasparras@mpa-garching.mpg.dehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/767Count likelihood evaluations2022-04-22T14:55:49ZPhilipp Arrasparras@mpa-garching.mpg.deCount likelihood evaluationsMartin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/766Port doc changes from NIFTy_7 to NIFTy_82022-04-12T14:05:05ZPhilipp Arrasparras@mpa-garching.mpg.dePort doc changes from NIFTy_7 to NIFTy_8Martin ReineckeMartin Reinecke