ift issueshttps://gitlab.mpcdf.mpg.de/groups/ift/-/issues2017-05-16T21:20:26Zhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/125pyfftw tests are not being run2017-05-16T21:20:26ZMartin Reineckepyfftw tests are not being runI just noticed that continuous integration does not run pyfftw tests, even when the package should be available. This accounts for the 2400 tests that are currently skipped in the pipelines.
I'm not sure why this happens. The relevant te...I just noticed that continuous integration does not run pyfftw tests, even when the package should be available. This accounts for the 2400 tests that are currently skipped in the pipelines.
I'm not sure why this happens. The relevant tests in `test_fft_operator.py` are guarded by
```
if module == "fftw" and "pyfftw" not in di:
raise SkipTest
```
which looks correct to me.https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/124Use sphinx for automated build of documentation2018-02-15T11:22:18ZTheo SteiningerUse sphinx for automated build of documentationIn the past I was successful with sphinx + napoleon
http://www.sphinx-doc.org/en/stable/ext/napoleon.htmlIn the past I was successful with sphinx + napoleon
http://www.sphinx-doc.org/en/stable/ext/napoleon.htmlPhilipp FrankPhilipp Frankhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/123SmoothingOperator: Correct units for sigma if `log_distances=True`?2017-05-28T06:56:12ZTheo SteiningerSmoothingOperator: Correct units for sigma if `log_distances=True`?Should the sigma be in log-units, too, if `log_distances` is set to True? If yes, how?Should the sigma be in log-units, too, if `log_distances` is set to True? If yes, how?Martin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/122DirectSmoothingOperator shows unwanted behaviour on edges2017-05-28T06:56:12ZTheo SteiningerDirectSmoothingOperator shows unwanted behaviour on edgesBy fixing the normalization of the direct/brute-force smoother, the edges now show some instabilities:
``
from nifty import *
x = RGSpace(64, harmonic=True)
p = PowerSpace(x)
sm = SmoothingOperator(p, sigma = 1.)
sm.domain
f = Field(p, v...By fixing the normalization of the direct/brute-force smoother, the edges now show some instabilities:
``
from nifty import *
x = RGSpace(64, harmonic=True)
p = PowerSpace(x)
sm = SmoothingOperator(p, sigma = 1.)
sm.domain
f = Field(p, val=1.)
f.val[16] = 2
plotter = plotting.PowerPlotter()
plotter.title = 'test'
plotter.plot(sm(f))
``
-> The contributions to the edge pixels don't add up anymore. This is somehow the 'adjoint problem' of what has been solved by Martin's fix.
Is there any chance to fix this?
![image](/uploads/ebaac233e4a170769656d7be751dcaa2/image.png)Martin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/121Questions on FieldType and FieldArray2017-05-28T06:56:12ZMartin ReineckeQuestions on FieldType and FieldArrayIt seems we forgot to discuss (and document) FieldType and FieldArray...
- could they be merged into a single class? FieldType seems unused, except as a parent of FieldArray.
- FieldType::process() seems unused and seems rather carel...It seems we forgot to discuss (and document) FieldType and FieldArray...
- could they be merged into a single class? FieldType seems unused, except as a parent of FieldArray.
- FieldType::process() seems unused and seems rather careless about errors. Do we need to keep this?https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/120Import on OS X fails2017-05-28T06:56:12ZPumpe, Daniel (dpumpe)Import on OS X failsOn the last master branch the
from nifty import * fails as
In [1]: from nifty import *
---------------------------------------------------------------------------
ImportError Traceback (most recent cal...On the last master branch the
from nifty import * fails as
In [1]: from nifty import *
---------------------------------------------------------------------------
ImportError Traceback (most recent call last)
<ipython-input-1-5ba8763d8b3e> in <module>()
----> 1 from nifty import *
/Users/danielpumpe/CloudStation/dpumpe/construction/NIFTy_3/lib/python2.7/site-packages/ift_nifty-3.0.4-py2.7-macosx-10.12-x86_64.egg/nifty/__init__.py in <module>()
55 from sugar import *
56
---> 57 import plotting
/Users/danielpumpe/CloudStation/dpumpe/construction/NIFTy_3/lib/python2.7/site-packages/ift_nifty-3.0.4-py2.7-macosx-10.12-x86_64.egg/nifty/plotting/__init__.py in <module>()
3 from figures import *
4 from colormap import *
----> 5 from plotter import *
6
ImportError: No module named plotter
As the latest master branch works on Martins machine, it is likely that this is due to subtile differences in the filesystems of OS X compared to Linux. Hence please change the naming of plotter.py in plotter or the name of the directory in order to overcome confusion and import errors.
https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/119tests for spaces2017-08-17T23:25:49ZReimar H Leiketests for spacesCreate more tests for all spaces. Especially tests for content are needed, that tests if the methods of spaces do what they should do on a numerical level.Create more tests for all spaces. Especially tests for content are needed, that tests if the methods of spaces do what they should do on a numerical level.https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/118Physical Units for demos2017-05-28T06:56:12ZReimar H LeikePhysical Units for demosVerify that the demo correctly applies units, such that if the resolution is doubled the physical problem stays the same.Verify that the demo correctly applies units, such that if the resolution is doubled the physical problem stays the same.Reimar H LeikeReimar H Leike2017-05-19https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/117Make "master" the default branch of the repository2017-05-28T06:56:12ZMartin ReineckeMake "master" the default branch of the repositoryThe Nifty_1 branch is (or at least should be) more or less unused by now. I suggest making "master" the default branch.The Nifty_1 branch is (or at least should be) more or less unused by now. I suggest making "master" the default branch.https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/116Allow plots of the sphere without depending on healpy2017-05-28T06:56:12ZMartin ReineckeAllow plots of the sphere without depending on healpyIt should not be a big effort to provide basic plotting functionality for functions on the sphere without having to import healpy. I will take care of this.It should not be a big effort to provide basic plotting functionality for functions on the sphere without having to import healpy. I will take care of this.Martin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/115Remove "Makefile"2017-05-28T06:56:12ZMartin ReineckeRemove "Makefile"Is this file needed for anything? It has not been updated for two years and seems disconnected from the rest of the package.Is this file needed for anything? It has not been updated for two years and seems disconnected from the rest of the package.https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/114implement PowerSpace nbins>dim check2017-05-28T06:56:12ZReimar H Leikeimplement PowerSpace nbins>dim checkPowerSpace should raise an exception if nbins is bigger than the number of dimensionsPowerSpace should raise an exception if nbins is bigger than the number of dimensionsReimar H LeikeReimar H Leikehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/113Simplify direct smoothing code2017-05-28T06:56:12ZMartin ReineckeSimplify direct smoothing codeIf I understood correctly, this functionality performs Gaussian smoothing on a (non-equidistant) array of data.
I can implement this efficiently and without using Cython; I just need some help in the case that the array is distributed o...If I understood correctly, this functionality performs Gaussian smoothing on a (non-equidistant) array of data.
I can implement this efficiently and without using Cython; I just need some help in the case that the array is distributed over several MPI tasks...
Implementing this would get rid of Nifty's Cython dependence.Martin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/112adjoint_inverse_times vs. inverse_adjoint_times2017-05-28T06:56:12ZTheo Steiningeradjoint_inverse_times vs. inverse_adjoint_timesIf a (left&right)-inverse of an operator exists, both methods are the same.If a (left&right)-inverse of an operator exists, both methods are the same.https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/111Unitary vs. Isometric&Coisometric2017-05-28T06:56:12ZTheo SteiningerUnitary vs. Isometric&CoisometricSHT are in general not unitary but they can be (co)isometric. In this case, there exist left- and right-inverses. Do we want to distinguish unitary -> (co)isometric in order to automatically enable e.g. `inverse_times` for certain isomet...SHT are in general not unitary but they can be (co)isometric. In this case, there exist left- and right-inverses. Do we want to distinguish unitary -> (co)isometric in order to automatically enable e.g. `inverse_times` for certain isometric SHTs?https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/110RGSpace hermitian decomposition bug for zerocentered & odd-number of pixels2017-05-28T06:56:12ZTheo SteiningerRGSpace hermitian decomposition bug for zerocentered & odd-number of pixelsTheo SteiningerTheo Steiningerhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/109Discuss default dtype problems for FFT/SHT Operator2017-05-28T06:56:12ZTheo SteiningerDiscuss default dtype problems for FFT/SHT Operatorhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/108Docstring: FFTOperator2017-07-01T11:06:18ZTheo SteiningerDocstring: FFTOperatorMartin ReineckeMartin Reineckehttps://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/107Add NIFTy 3 virtualenv to prelude.2017-05-28T06:56:12ZTheo SteiningerAdd NIFTy 3 virtualenv to prelude.Pumpe, Daniel (dpumpe)Pumpe, Daniel (dpumpe)https://gitlab.mpcdf.mpg.de/ift/nifty/-/issues/106Clarify dot product on multiple spaces2017-05-09T21:38:21ZJakob KnollmuellerClarify dot product on multiple spaces