Better parameterization of APOSMM class, plus docstring

[jlnav]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.93%. Comparing base (6b54991) to head (4357173).

Additional details and impacted files

@@                            Coverage Diff                            @@
##           experimental/jlnav_plus_shuds_asktell    #1583      +/-   ##
=========================================================================
- Coverage                                  89.95%   89.93%   -0.03%     
=========================================================================
  Files                                         79       79              
  Lines                                       7915     7925      +10     
  Branches                                    1176     1178       +2     
=========================================================================
+ Hits                                        7120     7127       +7     
- Misses                                       615      618       +3     
  Partials                                     180      180              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[shuds13]

FIELDS = [
    "initial_sample_size",
    "sample_points",
    "localopt_method",
    "rk_const",
    "xtol_abs",
    "ftol_abs",
    "dist_to_bound_multiple",
    "max_active_runs",
]

u = gen_specs.setdefault("user", {})
temp = {}
for k in FIELDS:
    val = locals().get(k)
    if val is not None:
        temp[k] = val
u.update(temp)

or separate required v optional

[jlnav]

Maybe this works but grabbing values from locals() to prevent redundancy just looks odd to me. I can still implement this.

[shuds13]

Its just a way to convert a string to a function name. Better than typing them all out right?