spurious algorithm in diagnostic_algorithms
Bug #788620 reported by
Jonathan Bull
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Fluidity |
Fix Released
|
Low
|
Patrick Farrell |
Bug Description
Somehow this line crept in at the bottom of diagnostic_
tensor_
It needs to be removed as there is no such algorithm.
Changed in fluidity: | |
importance: | Undecided → Low |
Changed in fluidity: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
It looks like before I sorted the available diagnostics, it was called "vector_ squared_ time_averaged_ algorithm" (even though it was marked as a diagnostic for a tensor field). It was added in r2725 by Dimitrios with the message
------- ------- ------- ------- ------- ------- ------- ------- ----
revno: 2725
svn revno: 2769 (on /trunk)
committer: dp204
timestamp: Tue 2011-01-18 12:32:35 +0000
message:
new diagnostics: time averaged scalar, vector, scalar squared, vector squared (this is a tensor including cross-products) and vector times scalar (for evaluating stuff like \overline{uT}) fields. These are interpolated between adapts. They don't like checkpointing for now. Totally untested.
Now, if there was any test using it, the diamond_validation test would have failed, and so I conclude that it is not tested. If that is the case, we should just delete it. In the meantime, I'll rename it.