mgastner / cartogram-cpp

Cartogram generator in C++
MIT License
8 stars 4 forks source link

Flatten Density Speedup + Time Report Bug Fix #145

Closed nihalzp closed 10 months ago

nihalzp commented 2 years ago

Closes #138

adisidev commented 2 years ago

@nihalzp do you think you could change the last commit such that it is compatible with a dynamic grid?

nihalzp commented 2 years ago

Thank you for the review Prof. Gastner and Adi.

I think the current version is compatible with Adi's change. In Adi's pull request, in inset_state.cpp line 411, we can add the following lines to make the flux grid dynamic:

ref_to_fluxx_init()->allocate(lx_, ly_);
ref_to_fluxy_init()->allocate(lx_, ly_);
make_fftw_plans_for_flux();
nihalzp commented 10 months ago

The content of this PR has been merged to the 'merge-pr' branch.