Skip to content
Next Next commit
modifying C++ energy calculation to match split-op code.
  • Loading branch information
leios committed Oct 23, 2018
commit ccdb6898b82d7e4b60eba94c36245635808af26e
39 changes: 22 additions & 17 deletions contents/quantum_systems/code/c++/energy.cpp
Original file line number Diff line number Diff line change
@@ -1,50 +1,55 @@
#include <complex>
#include <vector>
#include <complex>
#include <cstring>

#include <fftw3.h>

void fft(std::vector<std::complex<double>> x, bool inverse) {
std::vector<std::complex<double>> y(x.size());
for (size_t i = 0; i < x.size(); i++) {
y.push_back(std::complex(0.0, 0.0));
}

void fft(std::vector<std::complex<double>> x, int n, bool inverse) {
std::complex<double> y[n];
Copy link
Contributor

@zsparal zsparal Oct 23, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is unfortunately not standard C++. The way to solve this issue is:

std::vector<std::complex<double>> y(x.size(), std::complex<double>(0.0, 0.0));

This way you don't need either the memset or the push_back loop

memset(y, 0, sizeof(y));

fftw_plan p;

p = fftw_plan_dft_1d(x.size(), reinterpret_cast<fftw_complex*>(x.data()),
reinterpret_cast<fftw_complex*>(y.data()),
(inverse ? FFTW_BACKWARD : FFTW_FORWARD), FFTW_ESTIMATE);
fftw_complex *in = reinterpret_cast<fftw_complex*>(x.data());
fftw_complex *out = reinterpret_cast<fftw_complex*>(y);

p = fftw_plan_dft_1d(n, in, out,
(inverse ? FFTW_BACKWARD : FFTW_FORWARD), FFTW_ESTIMATE);


fftw_execute(p);
fftw_destroy_plan(p);

for (size_t i = 0; i < x.size(); ++i) {
x[i] = y[i] / sqrt((double)x.size());
x[i] = y[i] / sqrt(static_cast<double>(n));
}
}

double calculate_energy(std::vector<std::complex<double>> wfc, std::vector<std::complex<double>> h_r,
std::vector<std::complex<double>> h_k, double dx, size_t size) {
double calculate_energy(std::vector<std::complex<double>> wfc,
std::vector<std::complex<double>> h_r,
std::vector<std::complex<double>> h_k,
double dx, size_t size) {
std::vector<std::complex<double>> wfc_k(wfc);
std::vector<std::complex<double>> wfc_c(size);
fft(wfc_k, false);
fft(wfc_k, size, false);

for (size_t i = 0; i < size; ++i) {
wfc_c.push_back(conj(wfc[i]));
wfc_c[i] = conj(wfc[i]);
}

std::vector<std::complex<double>> energy_k(size);
std::vector<std::complex<double>> energy_r(size);

for (size_t i = 0; i < size; ++i) {
energy_k.push_back(wfc_k[i] * h_k[i]);
energy_k[i] = wfc_k[i] * h_k[i];
}

fft(energy_k, true);
fft(energy_k, size, true);

for (size_t i = 0; i < size; ++i) {
energy_k[i] *= wfc_c[i];
energy_r.push_back(wfc_c[i] * h_r[i] * wfc[i]);
energy_r[i] = wfc_c[i] * h_r[i] * wfc[i];
}

double energy_final = 0;
Expand Down
2 changes: 1 addition & 1 deletion contents/quantum_systems/quantum_systems.md
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ This ultimately looks like this:
{% sample lang="c" %}
[import:29-, lang:"c_cpp"](code/c/energy.c)
{% sample lang="cpp" %}
[import:26-57, lang:"c_cpp"](code/c++/energy.cpp)
[import:29-62, lang:"c_cpp"](code/c++/energy.cpp)
{% sample lang="py" %}
[import:4-17, lang:"python"](code/python/energy.py)
{% endmethod %}
Expand Down