NIFTy merge requestshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests2022-09-21T22:53:26Zhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/787Draft: Cached pipeline2022-09-21T22:53:26ZPhilipp Arrasparras@mpa-garching.mpg.deDraft: Cached pipelinePhilipp Arrasparras@mpa-garching.mpg.dePhilipp Arrasparras@mpa-garching.mpg.dehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/698Draft: Add code formatter2022-05-30T13:30:02ZPhilipp Arrasparras@mpa-garching.mpg.deDraft: Add code formatterOpen questions:
- Which order makes most sense for CI?
- formatting check -> tests -> demos
- tests -> formatting check -> demos (I think I prefer this one)
- tests -> demos -> formatting checkOpen questions:
- Which order makes most sense for CI?
- formatting check -> tests -> demos
- tests -> formatting check -> demos (I think I prefer this one)
- tests -> demos -> formatting checkhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/390Downgrade f-strings to good old format strings2019-12-06T08:18:39ZGordian EdenhoferDowngrade f-strings to good old format stringsSwitch back to ordinary format strings instead of f-strings introduced in fc6e304ab8, i.e. make NIFTy work again on python < 3.6.Switch back to ordinary format strings instead of f-strings introduced in fc6e304ab8, i.e. make NIFTy work again on python < 3.6.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/838don't open new fig in every iteration2023-02-13T09:36:48ZJakob Rothdon't open new fig in every iterationCurrently, opimize_kl opens a new figure, when plotting the minisanity history, in every iteration of the optimisation (See this line in the code: https://gitlab.mpcdf.mpg.de/ift/nifty/-/blob/NIFTy_8/src/minimization/optimize_kl.py#L613)...Currently, opimize_kl opens a new figure, when plotting the minisanity history, in every iteration of the optimisation (See this line in the code: https://gitlab.mpcdf.mpg.de/ift/nifty/-/blob/NIFTy_8/src/minimization/optimize_kl.py#L613). After the minisanity plot the figure is cleared but not closed. Thus after an `optimize_kl` run with n iterations, there are n open matplotlib figures. If one calls `plt.show()` after `optimize_kl` n empty figures will open. Here is a small demo:
```
import nifty8 as ift
import matplotlib.pyplot as plt
import numpy as np
from mpi4py import MPI
comm = MPI.COMM_WORLD
master = comm.Get_rank() == 0
############ some nifty with optimize_kl ##############
sp = ift.RGSpace(1)
op = ift.makeOp(ift.full(sp, 1)).ducktape('blub')
d = ift.full(sp, 1.)
n = ift.ScalingOperator(sp, 0.1, np.float64)
lh = ift.GaussianEnergy(d, inverse_covariance=n.inverse) @ op
n_iterations = 3
n_samples = 2
ic_sampling = ift.AbsDeltaEnergyController(name="Sampling (linear)",
deltaE=0.05, iteration_limit=100)
ic_newton = ift.AbsDeltaEnergyController(name='Newton', deltaE=0.5,
convergence_level=2, iteration_limit=35)
ic_sampling_nl = ift.AbsDeltaEnergyController(name='Sampling (nonlin)',
deltaE=0.5, iteration_limit=15,
convergence_level=2)
minimizer = ift.NewtonCG(ic_newton)
minimizer_sampling = ift.NewtonCG(ic_sampling_nl)
samples = ift.optimize_kl(lh, n_iterations, 2,
minimizer, ic_sampling, minimizer_sampling,
output_directory="folder",
comm=comm, plot_energy_history=True,
plot_minisanity_history=True)
#######################################################
x = np.linspace(1,10,100)
plt.plot(x,x**2)
if master:
plt.show()
```
Is there a reason to create a new figure every time? I guess this is a bug. I have removed the `plt.figure()`. Note: also in the energy history we don't have a `plt.figure()`.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/596Do not shadow native `dir`; rename to `direction`2021-02-03T07:54:08ZGordian EdenhoferDo not shadow native `dir`; rename to `direction`https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/822DOFDistributor: Fix `Field` ref in doc-string2023-01-02T09:40:08ZGordian EdenhoferDOFDistributor: Fix `Field` ref in doc-stringGordian EdenhoferGordian Edenhoferhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/84Document FFTOperator and rename its "inverse_times" to "adjoint_times"2017-05-09T23:46:25ZMartin ReineckeDocument FFTOperator and rename its "inverse_times" to "adjoint_times"This changes the interface of the FFTOperator according to today's discussions.
Documentation was added; tests have been adjusted.
NOTE: after this change, the Wiener filter demo no longer works, since it calls "inverse_times" for an F...This changes the interface of the FFTOperator according to today's discussions.
Documentation was added; tests have been adjusted.
NOTE: after this change, the Wiener filter demo no longer works, since it calls "inverse_times" for an FFTOperator.
Also, the SmoothingOperator no longer works, since it also requires the inverse FFT to work. I expect that it should not be allowed to work on spherical spaces anyway. Let's discuss this tomorrow.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/303Documentation feedback2019-04-10T08:07:59ZPhilipp Arrasparras@mpa-garching.mpg.deDocumentation feedbackhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/118Docstrings field2017-05-15T23:32:52ZTheo SteiningerDocstrings fieldhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/117Docstrings energy minimization2017-05-18T07:14:22ZTheo SteiningerDocstrings energy minimizationhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/100Docstring operators2017-05-12T23:00:37ZPumpe, Daniel (dpumpe)Docstring operatorsDocstrings for the Operators.Docstrings for the Operators.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/238Docstring improvements2018-04-04T12:38:30ZMartin ReineckeDocstring improvementshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/283docs: sync intro/teaser text from docs to readme2019-02-01T05:59:19ZLukas Platzdocs: sync intro/teaser text from docs to readmereadme.md still had the old intro text, now the same as in the docsreadme.md still had the old intro text, now the same as in the docshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/363docs/source/volume.rst: Explicit HT convention2019-10-28T19:52:29ZGordian Edenhoferdocs/source/volume.rst: Explicit HT conventionExplicitly state the convention used for the harmonic transform
operators and clearly state the difference between the convention used
in HEALPix and NIFTy.Explicitly state the convention used for the harmonic transform
operators and clearly state the difference between the convention used
in HEALPix and NIFTy.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/281Docs: New NIFTy gallery link2019-02-01T05:57:36ZLukas PlatzDocs: New NIFTy gallery linkUpdates the link to the new gallery.
Please check it actually works before merging :)Updates the link to the new gallery.
Please check it actually works before merging :)https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/678Docs CorrelatedField slice to average2021-08-22T11:13:10ZPhilipp FrankDocs CorrelatedField slice to averagehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/336Different scaling for y axis in power spectrum plots2019-07-31T08:26:52ZPhilipp Arrasparras@mpa-garching.mpg.deDifferent scaling for y axis in power spectrum plotsDoes not depend on zero mode anymore which is not plotted anywaysDoes not depend on zero mode anymore which is not plotted anywayshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/510Deterministic MPI KL2020-05-27T10:28:31ZMartin ReineckeDeterministic MPI KLThis patch tries to carry out summations over samples in a completely deterministic way, independent of the number of MPI tasks involved. My first tests with EHT indicate that this works, i.e. we get bit-identical results for the same in...This patch tries to carry out summations over samples in a completely deterministic way, independent of the number of MPI tasks involved. My first tests with EHT indicate that this works, i.e. we get bit-identical results for the same initial conditions when varying the number of tasks.
@reimar I'm not absolutely sure that we have a test for `_metric_sample`. How hard would it be to add one?https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/620Density estimator sugar2021-05-19T07:29:07ZGordian EdenhoferDensity estimator sugarMartin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/684density_estimator: Handle an optional CF prefix2021-09-14T11:59:31ZGordian Edenhoferdensity_estimator: Handle an optional CF prefixAccepting a prefix allows using the density estimator within other
models without having to worry about conflicting keys.Accepting a prefix allows using the density estimator within other
models without having to worry about conflicting keys.