- 07 Nov, 2020 3 commits
-
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
Also included it in the tarball
-
Goran Jelic-Cizmek authored
The effect of nonlinearities becomes important at small scales anyway, so we may as well start from 5 Mpc/h.
-
- 06 Nov, 2020 3 commits
-
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
-
- 05 Nov, 2020 1 commit
-
-
Goran Jelic-Cizmek authored
After hours of debugging, I finally figured out how to include everything in the tarball when running `make distcheck`. As shown here: https://stackoverflow.com/a/11966104 the key is to use $(srcdir), which denotes the top level directory. Also, I'm not sure what's the difference between `dist_pkgdata_DATA` and `EXTRA_DIST`, since both seem to get included anyway (maybe one is an install target and the other isn't? Idk). In any case, `make distcheck` should now work, with config files AND test data included, even though some of the tests will fail since I've used the CUBA library to generate those, and CUBA is much faster and more accurate for multidimensional integration (of course, if the user installs COFFE with CUBA enabled, then the tests will pass when running `make check`).
-
- 04 Nov, 2020 1 commit
-
-
Goran Jelic-Cizmek authored
Okay, this was very annoying to debug; basically, the flag `flatsky_density_lensing` turned on just the flat-sky approximation for density and lensing, and I wanted it to turn on all of the local-nonlocal terms (so RSD-lensing and d1-lensing as well, though they're zero). This commit adds that, AND renames the flag into `flatsky_local_nonlocal`. The results from the COFFE 2 paper remain valid, since the difference is miniscule. Also renamed `flatsky_lensing_lensing` into `flatsky_nonlocal`, since I may implement the other integrated terms at some point in the future, and added more detailed comments in the settings about what they actually do. Finally, left the names `functions_flatsky_[density|lensing]_lensing` intact for now, so just the flags in `coffe_parameters_t` have been changed.
-
- 03 Nov, 2020 1 commit
-
-
Goran Jelic-Cizmek authored
-
- 02 Nov, 2020 15 commits
-
-
Goran Jelic-Cizmek authored
"Warning" is now big scary "WARNING"
-
Goran Jelic-Cizmek authored
Default multipoles had double type instead if int
-
Goran Jelic-Cizmek authored
Removed checks for `input_separations` and `input_power_spectrum` since they make no difference now that COFFE does the check on its own.
-
Goran Jelic-Cizmek authored
It's too long, and unclear; renamed to flatsky_local, because it turns on the flat-sky approximation for all local terms.
-
Goran Jelic-Cizmek authored
Also, more default separations
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
Uses defaults now, not sure if it's the best solution, but it works.
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
Didn't test compiling without Cuba, so some errors appeared, should be fixed now.
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
Provided clearer explanations. Also removed matter_bias_analytic for now.
-
- 01 Nov, 2020 5 commits
-
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
`cubacores` used to accept ints, now it accepts pointers to ints, and the mismatch caused segfaults with the old code.
-
Goran Jelic-Cizmek authored
Also added a `approx_equal_const_epsilon` function, which uses as defaults COFFE_REL_EPSILON and COFFE_ABS_EPSILON, respectively, while `approx_equal` now takes custom values.
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
Option `only_cross_correlations` controls whether or not we should only consider the cross-correlations when computing the estimator/signal.
-
- 28 Oct, 2020 1 commit
-
-
Goran Jelic-Cizmek authored
-
- 26 Oct, 2020 7 commits
-
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
It's a bit inconsistent to include it in signal.c. Also, more comments on the various factors
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
Mostly casting things
-
Goran Jelic-Cizmek authored
Now there's just two, with self-explanatory purposes: - coffe_integrate_1d - coffe_integrate_multidimensional This reduces the need to copy-paste too much stuff everywhere.
-
Goran Jelic-Cizmek authored
signal_integrate_gsl is now called signal_integrate_multidimensional, which can be used to integrate any multidimensional function.
-
- 25 Oct, 2020 3 commits
-
-
Goran Jelic-Cizmek authored
-
Goran Jelic-Cizmek authored
TODO add all of those which we need when using the analytic method for density-lensing
-
Goran Jelic-Cizmek authored
Extra comma caused compilation errors
-