{"action":"created","issue":{"url":"https://api.github.com/repos/MikaelSlevinsky/FastTransforms.jl/issues/16","repository_url":"https://api.github.com/repos/MikaelSlevinsky/FastTransforms.jl","labels_url":"https://api.github.com/repos/MikaelSlevinsky/FastTransforms.jl/issues/16/labels{/name}","comments_url":"https://api.github.com/repos/MikaelSlevinsky/FastTransforms.jl/issues/16/comments","events_url":"https://api.github.com/repos/MikaelSlevinsky/FastTransforms.jl/issues/16/events","html_url":"https://github.com/MikaelSlevinsky/FastTransforms.jl/issues/16","id":229709213,"number":16,"title":"Spherical Harmonic bugs/enhancements","user":{"login":"MikaelSlevinsky","id":8176037,"avatar_url":"https://avatars0.githubusercontent.com/u/8176037?v=4","gravatar_id":"","url":"https://api.github.com/users/MikaelSlevinsky","html_url":"https://github.com/MikaelSlevinsky","followers_url":"https://api.github.com/users/MikaelSlevinsky/followers","following_url":"https://api.github.com/users/MikaelSlevinsky/following{/other_user}","gists_url":"https://api.github.com/users/MikaelSlevinsky/gists{/gist_id}","starred_url":"https://api.github.com/users/MikaelSlevinsky/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MikaelSlevinsky/subscriptions","organizations_url":"https://api.github.com/users/MikaelSlevinsky/orgs","repos_url":"https://api.github.com/users/MikaelSlevinsky/repos","events_url":"https://api.github.com/users/MikaelSlevinsky/events{/privacy}","received_events_url":"https://api.github.com/users/MikaelSlevinsky/received_events","type":"User","site_admin":false},"labels":[{"id":322755928,"url":"https://api.github.com/repos/MikaelSlevinsky/FastTransforms.jl/labels/bug","name":"bug","color":"fc2929","default":true},{"id":322755930,"url":"https://api.github.com/repos/MikaelSlevinsky/FastTransforms.jl/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-05-18T15:13:21Z","updated_at":"2018-02-05T16:10:41Z","closed_at":null,"author_association":"OWNER","body":"This issue is a placeholder for spherical harmonic bugs/enhancements that will be part of a patch:\r\n\r\n- [x] the way the butterfly algorithm is currently implemented, it only works for dimensions that are powers of two, though nothing stops this from working for general dimensions. This afflicts `FastSphericalHarmonicPlan` and `ThinSphericalHarmonicPlan`. For the time being, it's recommended to just pad with zeros to the `nextpow2`.\r\n\r\n- [x] the commands `sph2fourier` and `fourier2sph` should work on different band limits in `l` and `m`.\r\n\r\n- [ ] the command `plan_sph2fourier` is not type-stable as it very crudely dispatches to a different `SphericalHarmonicPlan` based on the bandlimit. This is negligible compared to pre-computation costs for large degrees (and is somewhat likened to Base's `factorize`).\r\n\r\n- [x] the convenience constructors `sphrand`, `sphrandn`, etc... take `m` and `n` and create an array with dimensions `m` and `2n-1` which is arguably misleading.\r\n\r\n- [x] it would be great to have an allocation-free `FFTW` override for converting the bivariate Fourier array to function values on the sphere at tensor product grids equispaced in angle. (Added starting with 67a6b560735ef8f82e1c6a7f82dad3e1a843fc8e)\r\n\r\n- [ ] add methods for complex data,\r\n\r\n- [ ] and, `NUFFT` variants for nonuniform point."},"comment":{"url":"https://api.github.com/repos/MikaelSlevinsky/FastTransforms.jl/issues/comments/363132432","html_url":"https://github.com/MikaelSlevinsky/FastTransforms.jl/issues/16#issuecomment-363132432","issue_url":"https://api.github.com/repos/MikaelSlevinsky/FastTransforms.jl/issues/16","id":363132432,"user":{"login":"MikaelSlevinsky","id":8176037,"avatar_url":"https://avatars0.githubusercontent.com/u/8176037?v=4","gravatar_id":"","url":"https://api.github.com/users/MikaelSlevinsky","html_url":"https://github.com/MikaelSlevinsky","followers_url":"https://api.github.com/users/MikaelSlevinsky/followers","following_url":"https://api.github.com/users/MikaelSlevinsky/following{/other_user}","gists_url":"https://api.github.com/users/MikaelSlevinsky/gists{/gist_id}","starred_url":"https://api.github.com/users/MikaelSlevinsky/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MikaelSlevinsky/subscriptions","organizations_url":"https://api.github.com/users/MikaelSlevinsky/orgs","repos_url":"https://api.github.com/users/MikaelSlevinsky/repos","events_url":"https://api.github.com/users/MikaelSlevinsky/events{/privacy}","received_events_url":"https://api.github.com/users/MikaelSlevinsky/received_events","type":"User","site_admin":false},"created_at":"2018-02-05T16:10:41Z","updated_at":"2018-02-05T16:10:41Z","author_association":"OWNER","body":"Another potential improvement to add to the list is to transpose the data. Givens rotations have better memory localization in row-major ordering, and the 1D row FFTs wouldn't need to transpose if they were 1D column FFTs."}}
{"actor":{"display_login":"MikaelSlevinsky"}}