Невозможно получить исходное изображение, используя FFT с FFTW

Question

Я использую FFTW в моем C-коде, и у меня есть некоторые проблемы. Во-первых, я могу преобразовать исходное изображение в два изображения (mag + phase) и вернуть исходное изображение с помощью обратного преобразования. Однако, если я хочу получить магнитный файл по центру по частоте, он больше не работает: у окончательного изображения есть некоторые проблемы.

Вот некоторые фрагменты моего кода. Может ли кто-нибудь помочь мне найти ошибку в моем коде?

РЕДАКТИРОВАТЬ: я исправил код, чтобы следовать рекомендации @francis, но мои проблемы всегда здесь.

enum {
    TYPE_CENTERED,
    TYPE_REGULAR
};

static void fft_to_spectra(fits* fit, fftw_complex *frequency_repr, double *as,
        double *ps, int nbdata) {
    unsigned int i;

    for (i = 0; i < nbdata; i++) {
        double r = creal(frequency_repr[i]);
        double im = cimag(frequency_repr[i]);
        as[i] = hypot(r, im);
        ps[i] = atan2(im, r);
    }
}

static void fft_to_freq(fits* fit, fftw_complex *frequency_repr, double *as, double *ps, int nbdata) {
    unsigned int i;

    for (i = 0; i < nbdata; i++) {
        frequency_repr[i] = as[i] * (cos(ps[i]) + I * sin(ps[i]));
    }
}

void change_symmetry(unsigned int width, unsigned int height, unsigned int i, unsigned int j, unsigned int *x,
        unsigned int *y) {

    if (i < width / 2 && j < height / 2) {
        *x = i + width / 2;
        *y = j + height / 2;
    }
    if (i >= width / 2 && j < height / 2) {
        *x = i - width / 2;
        *y = j + height / 2;
    }
    if (i < width / 2 && j >= height / 2) {
        *x = i + width / 2;
        *y = j - height / 2;
    }
    if (i >= width / 2 && j >= height / 2) {
        *x = i - width / 2;
        *y = j - height / 2;
    }
}

static void centered(WORD *buf, unsigned int width,
        unsigned int height) {
    unsigned int i, j;

    WORD *temp = malloc(width * height * sizeof(WORD));
    for (j = 0; j < height; j++) {
        for (i = 0; i < width; i++) {
            unsigned int x = i;
            unsigned int y = j;
            change_symmetry(width, height, i, j, &x, &y);

            temp[j * width + i] = buf[y * width + x];
        }
    }

    memcpy(buf, temp, sizeof(WORD) * width * height);
    free(temp);
}

static void normalisation_spectra(unsigned int w, unsigned int h, double *modulus, double *phase,
        WORD *abuf, WORD *pbuf) {
    unsigned int i;

    for (i = 0; i < h * w; i++) {
        pbuf[i] = round_to_WORD(((phase[i] + M_PI) * USHRT_MAX_DOUBLE / (2 * M_PI)));
        abuf[i] = round_to_WORD((modulus[i] / w / h));
    }
}

static void save_dft_information_in_gfit(fits *fit) {
    strcpy(gfit.dft.ord, fit->dft.type);
    strcpy(gfit.dft.ord, fit->dft.ord);
}

static void FFTD(fits *fit, fits *x, fits *y, int type_order, int layer) {
    WORD *xbuf = x->pdata[layer];
    WORD *ybuf = y->pdata[layer];
    WORD *gbuf = fit->pdata[layer];
    unsigned int i;
    unsigned int width = fit->rx, height = fit->ry;
    int nbdata = width * height;

    fftw_complex *spatial_repr = fftw_malloc(sizeof(fftw_complex) * nbdata);
    if (!spatial_repr) {
        return;
    }
    fftw_complex *frequency_repr = fftw_malloc(sizeof(fftw_complex) * nbdata);
    if (!frequency_repr) {
        fftw_free(spatial_repr);
        return;
    }

    /* copying image selection into the fftw data */
#ifdef _OPENMP
#pragma omp parallel for num_threads(com.max_thread) private(i) schedule(static) if(nbdata > 15000)
#endif
    for (i = 0; i < nbdata; i++) {
        spatial_repr[i] = (double) gbuf[i];
    }

    /* we run the Fourier Transform */
    fftw_plan p = fftw_plan_dft_2d(height, width, spatial_repr, frequency_repr,
            FFTW_FORWARD, FFTW_ESTIMATE);
    fftw_execute(p);

    /* we compute modulus and phase */
    double *modulus = malloc(nbdata * sizeof(double));
    double *phase = malloc(nbdata * sizeof(double));

    fft_to_spectra(fit, frequency_repr, modulus, phase, nbdata);

    //We normalize the modulus and the phase
    normalisation_spectra(width, height, modulus, phase, xbuf, ybuf);
    if (type_order == TYPE_CENTERED) {
        strcpy(x->dft.ord, "CENTERED");
        centered(xbuf, width, height);
        centered(ybuf, width, height);
    }

    free(modulus);
    free(phase);
    fftw_destroy_plan(p);
    fftw_free(spatial_repr);
    fftw_free(frequency_repr);
}

static void FFTI(fits *fit, fits *xfit, fits *yfit, int type_order, int layer) {
    WORD *xbuf = xfit->pdata[layer];
    WORD *ybuf = yfit->pdata[layer];
    WORD *gbuf = fit->pdata[layer];
    unsigned int i;
    unsigned int width = xfit->rx;
    unsigned int height = xfit->ry;
    int nbdata = width * height;

    double *modulus = calloc(1, nbdata * sizeof(double));
    double *phase = calloc(1, nbdata * sizeof(double));

    if (type_order == TYPE_CENTERED) {
        centered(xbuf, width, height);
        centered(ybuf, width, height);
    }

    for (i = 0; i < height * width; i++) {
        modulus[i] = (double) xbuf[i] * (width * height);
        phase[i] = (double) ybuf[i] * (2 * M_PI / USHRT_MAX_DOUBLE);
        phase[i] -= M_PI;
    }

    fftw_complex* spatial_repr = fftw_malloc(sizeof(fftw_complex) * nbdata);
    if (!spatial_repr) {
        return;
    }

    fftw_complex* frequency_repr = fftw_malloc(sizeof(fftw_complex) * nbdata);
    if (!frequency_repr) {
        fftw_free(spatial_repr);
        return;
    }

    fft_to_freq(fit, frequency_repr, modulus, phase, nbdata);

    fftw_plan p = fftw_plan_dft_2d(height, width, frequency_repr, spatial_repr,
            FFTW_BACKWARD, FFTW_ESTIMATE);
    fftw_execute(p);

    for (i = 0; i < nbdata; i++) {
        double pxl = creal(spatial_repr[i]) / nbdata;
        gbuf[i] = round_to_WORD(pxl);
    }

    free(modulus);
    free(phase);
    fftw_destroy_plan(p);
    fftw_free(spatial_repr);
    fftw_free(frequency_repr);
}

Здесь мои изображения, оригинал и FFTD (по центру) -> FFTI результат

Answer 1

План создается с использованием флага FFTW_MEASURE.Следовательно, несколько DFT вычисляются, и входной массив, вероятно, перезаписывается.Вот начало описания флагов планировщика в документации FFTW :

• FFTW_ESTIMATE указывает, что вместо фактических измерений различных алгоритмовпростая эвристика используется для быстрого выбора (возможно, неоптимального) плана. При этом флаге массивы ввода / вывода не перезаписываются во время планирования.

• FFTW_MEASURE говорит FFTW найти оптимизированный план, фактически вычисляя несколько БПФ и измеряявремя их исполнения.В зависимости от вашей машины это может занять некоторое время (часто несколько секунд).FFTW_MEASURE - опция планирования по умолчанию.

Либо переключитесь на FFTW_ESTIMATE, либо создайте план перед заполнением входного массива:

/* we run the Fourier Transform */
fftw_plan p = fftw_plan_dft_2d(width, height, spatial_repr, frequency_repr,
        FFTW_FORWARD, FFTW_MEASURE);

/* copying image selection into the fftw data */
#ifdef _OPENMP
#pragma omp parallel for num_threads(com.max_thread) private(i) schedule(static) if(nbdata > 15000)
#endif
for (i = 0; i < nbdata; i++) {
    spatial_repr[i] = (double) gbuf[i];
}

Если вы собираетесь использовать одно изображение, используя FFTW_ESTIMATE это путь.Напротив, если вы рассматриваете возможность обработки нескольких изображений, хорошим вариантом будет создание одного плана с использованием FFTW_MEASURE и его сохранение.Затем вы можете использовать Функции выполнения нового массива каждый раз, когда должно выполняться БПФ:

fftw_execute_dft(p, spatial_repr, frequency_repr);

Вы можете проверить возвращаемое значение malloc() или fftw_malloc(), чтобы проверить,распределение прошло правильно.Если нет, возвращается NULL.fftw_malloc() реализован как функция *X(kernel_malloc)(size_t n) в fftw-3.3.6-pl2 / kernel / kalloc.c.Он вызывает такие функции, как memalign() или _aligned_malloc() и другие.Оба эти возвращают NULL точно так же, как malloc() в случае сбоя.Наконец, я не обнаружил критической проблемы, связанной с выделением памяти для освобождения памяти в предоставленном вами фрагменте кода.

Аргумент double nbdata в fft_to_spectra(), вероятно, должен быть целым числом.Вальгринд мог бы посчитать это странным ...

РЕДАКТИРОВАТЬ: change_symmetry() следует изменить для нечетных размеров.Что-то вроде:

void change_symmetry_forward(unsigned int width, unsigned int height, unsigned int i, unsigned int j, unsigned int *x,
    unsigned int *y) {
     *x = i + width / 2;
     if (*x>=width){
           *x=*x-width;
     }
     *y = j + height / 2;
     if(*y>=height){
          *y =*y-height;
     }
}

и

void change_symmetry_backward(unsigned int width, unsigned int height, unsigned int i, unsigned int j, unsigned int *x,
    unsigned int *y) {
     *x = i +width- width / 2;
     if (*x>=width){
           *x=*x-width;
     }
     *y = j +height- height / 2;
     if(*y>=height){
          *y =*y-height;
     }
}