#2485: include complex and real scalar evolution code from Canuda in ET
Reporter: Roland Haas
Status: open
Milestone:
Version:
Type: task
Priority: major
Component:
Comment (by Taishi Ikeda):
> Do you mean “use\_jacobian = true“? If you have a look at line 254 of Scalar\_calc\_Tmunu.F90, you can see that there is an if statement that uses the combination "compute\_fluxes==1" and "use\_jacobian=false".
Oh, yes, sorry.. This is my typo. My point is that compute\_fluxes ==1 with use\_jacobian == true is not available. So, we should add comment saying “Flux can not be computed with multipatch“.
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2485/include-complex-a…
#2485: include complex and real scalar evolution code from Canuda in ET
Reporter: Roland Haas
Status: open
Milestone:
Version:
Type: task
Priority: major
Component:
Comment (by Giuseppe Ficarra):
ScalarEvolve:
> In current version, in Scalar\_calc\_Tmunu, "compute\_fluxes==1" and "use\_jacobian=false" is not available....Should we add the comments in param file ?
Do you mean “use\_jacobian = true“? If you have a look at line 254 of Scalar\_calc\_Tmunu.F90, you can see that there is an if statement that uses the combination "compute\_fluxes==1" and "use\_jacobian=false".
> in schedule, flux grid function is allocated, and Scalar\_zero\_densities is called when compute\_fluxes is true. But, in Scalar\_zero\_densities, flux grid functions are initialized only when use\_jacobian is false. So, if compute\_fluxes is true and use\_jacobian is true, flux grid function is not initialized.
fixed in [https://bitbucket.org/canuda/scalar/commits/cb03ec3b4d1bd2e32787ffbf2d32f4d…
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2485/include-complex-a…
#2629: MoL_PseudoEvolution vs ANALYSIS
Reporter: Gabriele Bozzola
Status: new
Milestone:
Version:
Type: bug
Priority: major
Component:
Comment (by Gabriele Bozzola):
Upon further reflection, and reading more carefully the previous message, my impression is that there’s no easy way to implement a `compute_every` function. I originally implemented the function following `WeylScal4`, which simply takes the modulo of the current iteration and the `compute_every` parameter. But, if I understand what is going on, this is not the correct thing to do, as I can see in my test runs \(unless `compute_every` is 1\).
So, should `compute_every` be removed from the `canuda` codes and `WeylScal4` ? \(Or at least, forced to be 1 for backwards compatibility\)
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2629/mol_pseudoevoluti…
#2629: MoL_PseudoEvolution vs ANALYSIS
Reporter: Gabriele Bozzola
Status: new
Milestone:
Version:
Type: bug
Priority: major
Component:
Comment (by Gabriele Bozzola):
It runs, but I am not sure it produces the correct results, so the PRs shouldn’t be merged at the moment.
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2629/mol_pseudoevoluti…