ift issueshttps://gitlab.mpcdf.mpg.de/groups/ift/-/issues2017-07-06T05:50:23Zhttps://gitlab.mpcdf.mpg.de/ift/D2O/-/issues/8Add support for `from array` indexing2017-07-06T05:50:23ZTheo SteiningerAdd support for `from array` indexingWhen building the kdict from pindex and kindex something of the following form must be done (a==kindex, b==pindex):
a = np.arange(16)*2
b = np.array([[3,2],[1,0]])
In [1]: a[b]
Out[1]:
array([[6, 4],
...When building the kdict from pindex and kindex something of the following form must be done (a==kindex, b==pindex):
a = np.arange(16)*2
b = np.array([[3,2],[1,0]])
In [1]: a[b]
Out[1]:
array([[6, 4],
[2, 0]])
Currently, this is solved using a hack:
p.apply_scalar_function(lambda z: obj[z])
This functionality could easily be added to the get_data interface.
Theo SteiningerTheo Steiningerhttps://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/D2O/-/issues/7The d2o_librarian will fail when mixing different MPI comms2017-07-06T05:50:23ZTheo SteiningerThe d2o_librarian will fail when mixing different MPI commsEvery local librarian instance on a node of a MPI cluster just increments its internal counter by one when a new d2o is registered. This gets out of sync, when only a part of the full cluster is covered by a special comm.
?Possible s...Every local librarian instance on a node of a MPI cluster just increments its internal counter by one when a new d2o is registered. This gets out of sync, when only a part of the full cluster is covered by a special comm.
?Possible solution: The individual librarians store the id of 'their' d2o and communicate a common id for their dictionary.
Con: Involves MPI communication.Theo SteiningerTheo Steiningerhttps://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/D2O/-/issues/6d2o cumsum and flatten rely on certain features of distribution strategy2017-07-06T05:50:23ZTheo Steiningerd2o cumsum and flatten rely on certain features of distribution strategycumsum and flatten assume: if the shape of the d2o changes through flattening, the distribution strategy was "slicing". cumsum and flatten assume: if the shape of the d2o changes through flattening, the distribution strategy was "slicing". Theo SteiningerTheo Steiningerhttps://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/D2O/-/issues/5Add function d2o.arange2017-07-06T05:50:23ZTheo SteiningerAdd function d2o.arangeTheo SteiningerTheo Steiningerhttps://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/D2O/-/issues/4Add unit tests for `copy=True/False` functionality2017-07-06T05:50:23ZTheo SteiningerAdd unit tests for `copy=True/False` functionalityTheo SteiningerTheo Steiningerhttps://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/D2O/-/issues/24Missing file in last commit?2018-03-18T10:55:45ZMartin ReineckeMissing file in last commit?Is it possible that you forgot to add a file `random.py` in your last commit?
As things are, D2O now imports Python's default `random` module, but this probably doesn't address the problems with MPI-parallel seeding you mentioned.Is it possible that you forgot to add a file `random.py` in your last commit?
As things are, D2O now imports Python's default `random` module, but this probably doesn't address the problems with MPI-parallel seeding you mentioned.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/D2O/-/issues/25Meta-issue: how to deal with open D2O issues?2018-03-20T14:57:13ZMartin ReineckeMeta-issue: how to deal with open D2O issues?@theos, @ensslint:
There are currently 22 open issues in D2O, and I don't expect that Theo will have the time to work on them. Unfortunately, no one else has the necessary knowledge.
Any suggestions how to proceed here?@theos, @ensslint:
There are currently 22 open issues in D2O, and I don't expect that Theo will have the time to work on them. Unfortunately, no one else has the necessary knowledge.
Any suggestions how to proceed here?https://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/107Add NIFTy 3 virtualenv to prelude.2017-05-28T06:56:12ZTheo SteiningerAdd NIFTy 3 virtualenv to prelude.Pumpe, Daniel (dpumpe)Pumpe, Daniel (dpumpe)