APOG_public/SP2XR
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: bug fixed in conversion from BC mass to diam due to density units mismatch in the config file and default values

Browse Source
This commit is contained in:
bertoz_b
2025-08-22 10:48:08 +02:00
parent bbd21ba7b9
commit d6b3f2028f
1 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/sp2xr/helpers.py
View File

@@ -34,8 +34,8 @@ def load_and_resolve_config(args):
"do_BC_hist": get(base, "workflow.BC_hist", False),
"do_scatt_hist": get(base, "workflow.scatt_hist", False),
"do_timelag_hist": get(base, "workflow.timelag_hist", False),
"rho_eff": choose(args.BC_rho, base, "bc.rho_eff", 1.8),
"BC_type": choose(args.BC_type, base, "bc.type", "constant_effective_density"),
"rho_eff": choose(args.BC_rho, base, "bc.rho_eff", None),
"BC_type": choose(args.BC_type, base, "bc.type", None),
"cluster": {
"use_local": choose(args.local, base, "cluster.use_local"),
"cores": choose(args.cores, base, "cluster.cores", 8),
@@ -177,14 +177,12 @@ def parse_args():
)
# BC proeprties
p.add_argument(
"--BC_rho", type=float, default=1.8, help="BC density (default: 1.8)"
)
p.add_argument("--BC_rho", type=float, default=None, help="BC density")
p.add_argument(
"--BC_type",
type=str,
default="constant_effective_density",
help="BC type (default: constant_effective_density)",
default=None,
help="BC type",
)
# cluster / resource knobs (all optional)