output debug plots to artifacts dir

experiments/AMIP/coupler_driver.jl

@@ -175,7 +175,7 @@ The data files are downloaded from the `ClimaCoupler` artifacts directory. If th
 original sources.
 =#
 
-debug_dir = joinpath(COUPLER_ARTIFACTS_DIR, "debug_output/")
+debug_dir = COUPLER_ARTIFACTS_DIR
 isdir(debug_dir) ? nothing : mkpath(debug_dir)
 
 # get the paths to the necessary data files: land-sea mask, sst map, sea ice concentration
@@ -599,21 +599,21 @@ The concrete steps for proper initialization are:
 
 # 1.coupler updates surface model area fractions
 update_surface_fractions!(cs)
-debug(cs, "debug_output/2_surface_fraction_update")
+debug(cs, debug_dir * "2_surface_fraction_update")
 
 # 2.surface density (`ρ_sfc`): calculated by the coupler by adiabatically extrapolating atmospheric thermal state to the surface.
 # For this, we need to import surface and atmospheric fields. The model sims are then updated with the new surface density.
 import_combined_surface_fields!(cs.fields, cs.model_sims, cs.boundary_space, cs.turbulent_fluxes)
 import_atmos_fields!(cs.fields, cs.model_sims, cs.boundary_space, cs.turbulent_fluxes)
 update_model_sims!(cs.model_sims, cs.fields, cs.turbulent_fluxes)
-debug(cs, "debug_output/3_update_sims")
+debug(cs, debug_dir * "3_update_sims")
 
 # 3.surface vapor specific humidity (`q_sfc`): step surface models with the new surface density to calculate their respective `q_sfc` internally
 ## TODO: the q_sfc calculation follows the design of the bucket q_sfc, but it would be neater to abstract this from step! (#331)
 step!(land_sim, Δt_cpl)
 step!(ocean_sim, Δt_cpl)
 step!(ice_sim, Δt_cpl)
-debug(cs, "debug_output/4_step_surface_models")
+debug(cs, debug_dir * "4_step_surface_models")
 
 # 4.turbulent fluxes: now we have all information needed for calculating the initial turbulent surface fluxes using the combined state
 # or the partitioned state method
@@ -632,11 +632,11 @@ elseif cs.turbulent_fluxes isa PartitionedStateFluxes
     CA.SurfaceConditions.update_surface_conditions!(atmos_sim.integrator.u, new_p, atmos_sim.integrator.t) ## sets T_sfc (but SF calculation not necessary - requires split functionality in CA)
     atmos_sim.integrator.p.precomputed.sfc_conditions .= new_p.precomputed.sfc_conditions
 end
-debug(cs, "debug_output/5_after_turb_fluxes")
+debug(cs, debug_dir * "5_after_turb_fluxes")
 
 # 5.reinitialize models + radiative flux: prognostic states and time are set to their initial conditions. For atmos, this also triggers the callbacks and sets a nonzero radiation flux (given the new sfc_conditions)
 reinit_model_sims!(cs.model_sims)
-debug(cs, "debug_output/6_after_reinit")
+debug(cs, debug_dir * "6_after_reinit")
 
 # 6.update all fluxes: coupler re-imports updated atmos fluxes (radiative fluxes for both `turbulent_fluxes` types
 # and also turbulent fluxes if `turbulent_fluxes isa CombinedStateFluxes`,
@@ -645,7 +645,7 @@ debug(cs, "debug_output/6_after_reinit")
 import_atmos_fields!(cs.fields, cs.model_sims, cs.boundary_space, cs.turbulent_fluxes)
 update_model_sims!(cs.model_sims, cs.fields, cs.turbulent_fluxes)
 
-debug(cs, "debug_output/7_import_atmos_fluxes")
+debug(cs, debug_dir * "/7_import_atmos_fluxes")
 
 #=
 ## Coupling Loop
@@ -784,7 +784,7 @@ end #hide
 ## run the coupled simulation
 solve_coupler!(cs);
 
-debug(cs, "debug_output/6_after_solve")
+debug(cs, debug_dir * "6_after_solve")
 #=
 ## Postprocessing
 Currently all postprocessing is performed using the root process only.

experiments/AMIP/user_io/debug_plots.jl

@@ -12,8 +12,8 @@ using ClimaCore
 Plot the fields of a coupled simulation and save plots to a directory.
 """
 function debug(cs::CoupledSimulation, dir = "debug", cs_fields_ref = nothing)
-    mkpath(dir)
-    @info "plotting debug in " * dir
+    # mkpath(dir)
+    # @info "plotting debug in " * dir
     for sim in cs.model_sims
         debug(sim, dir)
     end
@@ -80,7 +80,7 @@ function debug(cs_fields::NamedTuple, dir, cs_fields_ref = nothing)
         end
     end
     Plots.plot(all_plots..., size = (1500, 800))
-    Plots.png(joinpath(dir, "debug_coupler"))
+    Plots.png(dir * "debug_coupler")
 
     # plot anomalies if a reference cs.fields, `cs_fields_ref`, are provided
     if !isnothing(cs_fields_ref)
@@ -99,7 +99,7 @@ function debug(cs_fields::NamedTuple, dir, cs_fields_ref = nothing)
             )
         end
         Plots.plot(all_plots..., size = (1500, 800))
-        Plots.png(joinpath(dir, "debug_coupler_amomalies"))
+        Plots.png(dir * "debug_coupler_amomalies")
     end
 end
 
@@ -149,14 +149,12 @@ function debug(sim::ComponentModelSimulation, dir)
             nz = nz,
         )
         cpu_field = ClimaCore.Fields.ones(cpu_space)
-        @show space
-        @show cpu_space
         parent(cpu_field) .= parent(field)
 
         push!(all_plots, Plots.plot(cpu_field, title = string(field_name) * print_extrema(field)))
     end
     fig = Plots.plot(all_plots..., size = (1500, 800))
-    Plots.png(joinpath(dir, "debug_$(name(sim))"))
+    Plots.png(dir * "debug_$(name(sim))")
 
 end