2013-11-21 12 views
6

Czy istnieje skuteczny sposób zmiany układu współrzędnych kartezjańskich na n-spherical one? Transformacja jest następujący: enter image description hereUkład współrzędnych n-kuli do układu współrzędnych kartezjańskich

Oto mój kod, ale chcę, aby pozbyć się pętli:

import numpy as np 
import scipy.sparse 

    def coord_transform_n(r,alpha): 
     """alpha: the n-2 values between [0,\pi) and last one between [0,2\pi) 
     """ 
     x=[] 
     for i in range(alpha.shape[0]): 
      x.append(r*np.prod(np.sin(alpha[0:i]))*np.cos(alpha[i])) 
     return np.asarray(x) 
    print coord_transform_n(1,np.asarray(np.asarray([1,2]))) 
+0

zdałem sobie sprawę, mój kod nie jest poprawny. Nie uwzględniam ostatniej współrzędnej, a mianowicie x_n! – Cupitor

Odpowiedz

6

Twój oryginalny kod może zostać przyspieszone z zapamiętywania pośredniego produktu sin, tj

def ct_dynamic(r, alpha): 
    """alpha: the n-2 values between [0,\pi) and last one between [0,2\pi) 
    """ 
    x = np.zeros(len(alpha) + 1) 
    s = 1 
    for e, a in enumerate(alpha): 
     x[e] = s*np.cos(a) 
     s *= np.sin(a) 
    x[len(alpha)] = s 
    return x*r 

Ale wciąż traci na prędkości do NumPy podejście oparte

def ct(r, arr): 
    a = np.concatenate((np.array([2*np.pi]), arr)) 
    si = np.sin(a) 
    si[0] = 1 
    si = np.cumprod(si) 
    co = np.cos(a) 
    co = np.roll(co, -1) 
    return si*co*r 

>>> n = 10 
>>> c = np.random.random_sample(n)*np.pi 
>>> all(ct(1,c) == ct_dynamic(1,c)) 
True 

>>> timeit.timeit('from __main__ import coord_transform_n as f, c; f(2.4,c)', number=10000) 
2.213547945022583 

>>> timeit.timeit('from __main__ import ct_dynamic as f, c; f(2.4,c)', number=10000) 
0.9227950572967529 

>>> timeit.timeit('from __main__ import ct as f, c; f(2.4,c)', number=10000) 
0.5197498798370361 
+0

Dziękuję bardzo. Zrozumiałem, że mój kod jest nieprawidłowy. Nie uwzględniam ostatniej współrzędnej, a mianowicie x_n! – Cupitor

+0

@Naji nie jest trudno to naprawić, wszystkie oceny już zrobione. Uaktualniłem swoją odpowiedź na poprawne implementacje, zostawię twoją;) – alko

+1

Tak, już zrobiłem. Po prostu cię powiadomiłem: D – Cupitor

4

Moja sugestia: Zestawić zatoki w jednym wektorze, korzystanie cumprod na niego, a następnie każdy z każdym swoim cosinusem.

Powiązane problemy