NIFTy merge requestshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests2017-07-12T08:02:57Zhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/147re-organize PowerSpace2017-07-12T08:02:57ZMartin Reineckere-organize PowerSpaceThis branch aims at making the current `PowerSpace` design simpler.
The following changes have been made so far:
- the `PowerSpace` members are always fully computed upon construction of the `PowerSpace`; caching has been removed. Th...This branch aims at making the current `PowerSpace` design simpler.
The following changes have been made so far:
- the `PowerSpace` members are always fully computed upon construction of the `PowerSpace`; caching has been removed. This still has performance drawbacks which I hope to address in the future.
- the parameters `logarithmic` and `nbin` can still be supplied to the constructor, but they are no longer part of the `PowerSpace`'s state. They are used to compute the `binbounds`, and forgotten afterwards. (In the current implementation it is possible to set these attributes to values which contradict the actual `binbounds`, which is inconsistent.)
- `pundex` is currently not supported, but can be reintroduced if needed.
- "natural" binning (which happens if neither `logarithmic`, `nbin` or `binbounds` are specified to the constructor) has been improved to avoid spurious extra bins due to numerical noise.
- index computation has been accelerated significantly
- the `config` property has been removed; `binbounds` is now a direct property of the class.
I'm sure that a lot will have to change before this can be merged; please consider this merge request mainly as a starting point for discussion!
As a quick demonstration of the advantages of the new implementation, run the following commands in both "master" and "index_games" branch and watch CPU time and memory consumption:
```
a=RGSpace((10000,10000),harmonic=True)
p=PowerSpace(a,binbounds=sqrt(2.)*np.arange(10000.))
```https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/133Add projection to field2017-05-29T13:47:43ZJakob KnollmuellerAdd projection to fieldadded projection of a Field into power space to Field class (mathematically: m^\dagger P m ) required for critical Filteringadded projection of a Field into power space to Field class (mathematically: m^\dagger P m ) required for critical Filteringhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/142Tiny fix for Field.power_synthesize() & RGSpace.hermitian_fixed_points()2017-05-27T14:07:22ZPumpe, Daniel (dpumpe)Tiny fix for Field.power_synthesize() & RGSpace.hermitian_fixed_points()Field.power_synthesize working again as, RGSpace.hermitian_fixed_spaces was uncommented.Field.power_synthesize working again as, RGSpace.hermitian_fixed_spaces was uncommented.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/134more docu on sqrt power spectrum issue #1282017-05-22T20:06:17ZReimar H Leikemore docu on sqrt power spectrum issue #128Added some clarifications to the documentation and fixed typosAdded some clarifications to the documentation and fixed typoshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/105Plotting2017-05-17T21:51:11ZTheo SteiningerPlottinghttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/130WIP: Docstring operators2017-05-17T21:50:59ZTheo SteiningerWIP: Docstring operatorshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/118Docstrings field2017-05-15T23:32:52ZTheo SteiningerDocstrings fieldhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/119WIP: store for later2017-05-15T18:57:55ZTheo SteiningerWIP: store for laterPlease don't merge! This is my temporary storage branch for odds and ends.Please don't merge! This is my temporary storage branch for odds and ends.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/108Unit2017-05-13T00:21:49ZTheo SteiningerUnithttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/101Test power space2017-05-11T14:22:01ZReimar H LeikeTest power spaceAdded consistency check for pundix pindex inversion and slightly clarified docstring for binboundsAdded consistency check for pundix pindex inversion and slightly clarified docstring for binboundshttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/81Unit2017-05-11T12:25:43ZTheo SteiningerUnitPumpe, Daniel (dpumpe)Pumpe, Daniel (dpumpe)https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/92Fix fft side effect2017-05-10T10:39:17ZTheo SteiningerFix fft side effecthttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/88Remove line energy2017-05-10T00:13:00ZTheo SteiningerRemove line energyhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/80WIP: Newton2017-05-06T23:50:14ZTheo SteiningerWIP: Newtonhttps://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/79Mmm52017-05-06T23:44:45ZTheo SteiningerMmm5https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/71Issue772017-04-26T12:08:57ZMartin ReineckeIssue77Try to fix issue77 (hashing/equality problems).Try to fix issue77 (hashing/equality problems).https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/61Small wiener filter toy test incl. units & small correction in create_power_o...2017-04-26T11:37:26ZPumpe, Daniel (dpumpe)Small wiener filter toy test incl. units & small correction in create_power_operatorHi,
Torsten asked me to incorporate a small Wiener filter example incl. units. Therefore I created a small ResponseOperator (including. convolution, exposure and domain/target space).
Numerical test showed me that the create_power_...Hi,
Torsten asked me to incorporate a small Wiener filter example incl. units. Therefore I created a small ResponseOperator (including. convolution, exposure and domain/target space).
Numerical test showed me that the create_power_operator does not give a proper PowerOperator (at least it does not match the results of comparable one in NIFTy_1). However it is only due to the incorrect incorporation of volume factors in the DiagonalOperator. The bar keyword has to be set to True.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/63Remove many dependencies, introduce dependency from pyHealpix, various cleanups2017-04-20T08:21:34ZMartin ReineckeRemove many dependencies, introduce dependency from pyHealpix, various cleanupsThis removes Nifty's dependencies from healpy, libsharp and many more and replaces them by a single dependency from the new pyHealpix package.
This branch also contains the changes from the 'strip_gfft' and 'reduce_cython' branches, so ...This removes Nifty's dependencies from healpy, libsharp and many more and replaces them by a single dependency from the new pyHealpix package.
This branch also contains the changes from the 'strip_gfft' and 'reduce_cython' branches, so it is sufficient just to merge this one branch.
Tests have been adjusted accordingly and many cleanups were made; I especially removed obsolete (and confusing) bits of documentation.
Sorry for not producing several smaller merge requests ... but given the current turnaround time for patch reviews this would have become nearly impossible.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/62Move some Cython code back to Python2017-04-20T08:21:23ZMartin ReineckeMove some Cython code back to PythonThis moves two Cython files back to Python ... I found implementations that are efficient enough to avoid the Cython detour.This moves two Cython files back to Python ... I found implementations that are efficient enough to avoid the Cython detour.https://gitlab.mpcdf.mpg.de/ift/nifty/-/merge_requests/58strip dependency of GFFT and simplify gfft_dummy.py2017-04-20T08:21:07ZMartin Reineckestrip dependency of GFFT and simplify gfft_dummy.py