Fluidity

Error checking for diagnostic field dependencies does not work when the diagnostic is dependent on a field that is hidden somewhere in the options tree

Bug #1026041 reported by Samuel Parkinson on 2012-07-18

This bug affects 2 people

Affects		Status	Importance	Assigned to	Milestone
	Fluidity	In Progress	Critical	Jon Hill

Bug Description

subroutine diagnostic_fields_new_check_options in Diagnostic_Fields_New.F90 checks that fields a diagnostic field depends upon exist in the options tree. However, it only looks for these fields in the root node of a material phase, often they may be hidden in the options tree, for example in parameterisations/k_epsilon/....

Related branches

lp:~jon-hill/fluidity/diagnostic_check

Ready for review for merging into lp:fluidity

Stephan Kramer: Needs Fixing on 2014-01-24

sam: Pending requested 2014-01-24

Jon Hill (jon-hill) on 2012-08-01

Changed in fluidity:
assignee:	nobody → Samuel Parkinson (s-parkinson11)
importance:	Undecided → Low

Jon Hill (jon-hill) on 2014-01-08

Changed in fluidity:
importance:	Low → Critical
assignee:	Samuel Parkinson (s-parkinson11) → Jon Hill (jon-hill)
status:	New → In Progress

Revision history for this message

Jon Hill (jon-hill) wrote on 2014-01-24:

I've added a new module that contains such hidden paths, so that populate_state and diagnostic::check_options can use these. I've also added a test to check that something hidden under parameterisation.prescribed_diffusivity can be used in a new diagnostic field.

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.