NIFTy merge requestshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests2017-10-06T00:28:42Zhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/203Demo tweaks2017-10-06T00:28:42ZMartin ReineckeDemo tweaksThis changes the default distribution strategy in all demos to "not" instead of "fftw", which should make them runnable on most people's computers.
Also some math calls are now explicitly calling numpy instead of the ift-specific versio...This changes the default distribution strategy in all demos to "not" instead of "fftw", which should make them runnable on most people's computers.
Also some math calls are now explicitly calling numpy instead of the ift-specific versions, in preparation for future changes.
log_normal_wiener_filter now works with the new minimizers.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/204Add PowerProjection operator; rewrite CriticalPowerEnergy2017-10-18T13:08:28ZMartin ReineckeAdd PowerProjection operator; rewrite CriticalPowerEnergyhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/205fix critical filter2017-11-28T17:31:20ZJakob Knollmuellerfix critical filterpower operator needs exp(t0), not ps0power operator needs exp(t0), not ps0https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/206added nonlinear_critical_filter in demo, fixed some things in energies, added…2017-12-15T12:03:24ZJakob Knollmuelleradded nonlinear_critical_filter in demo, fixed some things in energies, added…added nonlinear_critical_filter in demo, fixed some things in energies, added DOFSpace+Projection (untested)added nonlinear_critical_filter in demo, fixed some things in energies, added DOFSpace+Projection (untested)https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/207new operator convenience functionality2018-01-25T11:24:30ZMartin Reineckenew operator convenience functionalityThe operators on this branch have the following new properties:
- they can be chained with "*", "+", and "-"
- they have new properties ".inverse" and ".adjoint", which return the inverse resp. the adjoint operator
This allows very simp...The operators on this branch have the following new properties:
- they can be chained with "*", "+", and "-"
- they have new properties ".inverse" and ".adjoint", which return the inverse resp. the adjoint operator
This allows very simple creation of combined operators. See the end of `demos/wiener_filter_easy.py` or `nifty/library/wiener_filter_curvature.py` for a nice demo.
@kjako, @dpumpe, @parras, @theos, @reimar, @ensslint : I'd be happy about any feedback!Martin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/208WIP: Throw exception if curvature is not implemented for RelaxedNewton2018-01-16T17:21:37ZPhilipp Arrasparras@mpa-garching.mpg.deWIP: Throw exception if curvature is not implemented for RelaxedNewtonThe problem: If an energy class without curvature is defined and then minimized with RelaxedNewton, it does not raise an error and continues the script. In this way, I thought that my algorithm estimates noise variances but it doesn't.
...The problem: If an energy class without curvature is defined and then minimized with RelaxedNewton, it does not raise an error and continues the script. In this way, I thought that my algorithm estimates noise variances but it doesn't.
I do not know how to implement this check properly. Do I really need to import `sys` for that and use `sys.exit()`?https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/209WIP: Byebye volume factors2018-01-25T11:23:34ZMartin ReineckeWIP: Byebye volume factorsThis branch contains Reimar's work on eliminating (most of) the volume factors.
Unit tests are passing, but the demos need more adjusting.
Please test this as hard as you can! (And add new unit tests if appropriate...)
@reimar, @kjako, ...This branch contains Reimar's work on eliminating (most of) the volume factors.
Unit tests are passing, but the demos need more adjusting.
Please test this as hard as you can! (And add new unit tests if appropriate...)
@reimar, @kjako, @ensslint, @parras: this means you ;)Reimar H LeikeReimar H Leikehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/210Merge nightly branch into master2018-02-01T12:31:58ZMartin ReineckeMerge nightly branch into masterAs far as I know, the changes on the nightly branch have sufficiently matured to be merged onto master. I plan to do this on February 1. Please object if you still need the master branch in its current stage!
@theos, OK with you?As far as I know, the changes on the nightly branch have sufficiently matured to be merged onto master. I plan to do this on February 1. Please object if you still need the master branch in its current stage!
@theos, OK with you?Martin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/211Make nonlinearities available for user2018-01-26T09:30:01ZPhilipp Arrasparras@mpa-garching.mpg.deMake nonlinearities available for userhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/212WIP: Resolve "Gradients of all PowerEnergy classes and NoiseEnergy seem to be...2018-01-30T13:00:16ZPhilipp Arrasparras@mpa-garching.mpg.deWIP: Resolve "Gradients of all PowerEnergy classes and NoiseEnergy seem to be broken"Closes #214Closes #214https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/213WIP: Resolve "Tests: Energy class"2018-01-30T19:03:36ZPhilipp Arrasparras@mpa-garching.mpg.deWIP: Resolve "Tests: Energy class"Closes #99Closes #99https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/214WIP: Resolve "Tests: Energy class"2018-01-30T19:03:35ZPhilipp Arrasparras@mpa-garching.mpg.deWIP: Resolve "Tests: Energy class"Closes #99Closes #99https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/215Energy tests2018-01-30T20:37:21ZPhilipp Arrasparras@mpa-garching.mpg.deEnergy testsBe aware: I changed the interface to the energies. The energy classes do not take the keyword `fft` anymore. Instead use `ht` now since we want t encourage users to facilitate the `HarmonicTransformOperator` from now on.Be aware: I changed the interface to the energies. The energy classes do not take the keyword `fft` anymore. Instead use `ht` now since we want t encourage users to facilitate the `HarmonicTransformOperator` from now on.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/216Add adjointness and inverse test2018-02-05T12:33:34ZPhilipp Arrasparras@mpa-garching.mpg.deAdd adjointness and inverse testI propose to add a adjointness test and a test for the inverse to the `LinearOperator` base class. This might encourage users to use these tests more often.
TODO: If you like this feature, we should use this functionality in the tests...I propose to add a adjointness test and a test for the inverse to the `LinearOperator` base class. This might encourage users to use these tests more often.
TODO: If you like this feature, we should use this functionality in the tests and also in the demos.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/217Notebook demo2018-02-05T10:13:29ZMartin ReineckeNotebook demohttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/218Work on demos2018-02-05T14:48:25ZPhilipp Arrasparras@mpa-garching.mpg.deWork on demoshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/219Citations2018-03-18T11:44:11ZPhilipp Arrasparras@mpa-garching.mpg.deCitationsI would propse to also explain how to cite NIFTy with the help of BibTeX. To be honest the How-to-cite instruction is not too helpful after all if one uses BibTeX.
In the LaTeX document class `\documentclass[conference]{IEEEtran}` the...I would propse to also explain how to cite NIFTy with the help of BibTeX. To be honest the How-to-cite instruction is not too helpful after all if one uses BibTeX.
In the LaTeX document class `\documentclass[conference]{IEEEtran}` the bibliography entry looks like this when using my proposed BibTeX entry:
>>>
Martin Reinecke, Theo Steininger. NIFTy – Numerical
Information Field TheorY. Version nifty4. Feb. 5, 2018.
URL : https://gitlab.mpcdf.mpg.de/ift/NIFTy.
>>>
@ensslint, is this okay? Shall we change something?https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/220Add test for laplace operator2018-02-06T13:42:31ZPhilipp Arrasparras@mpa-garching.mpg.deAdd test for laplace operatorhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/221Minor spelling fix2018-02-06T15:09:50ZPhilipp Arrasparras@mpa-garching.mpg.deMinor spelling fixhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/222Fix demo2018-02-12T09:02:42ZPhilipp Arrasparras@mpa-garching.mpg.deFix demoAlles andere wäre auch zu schön gewesen :DAlles andere wäre auch zu schön gewesen :D