fstermann
commented
1 year ago LGTM! Bin nur nicht so ein großer Fan von qtest, finde es explizit fast immer besser
Ja stimmt, fand die den Output von format zu nutzen zu gut weil das direkt auf qtest passt :D Aber dabei ist mir auch aufgefallen das wir aufpassen müssen falls jemand die Range überschreiben will, also
a <- p_int(0, 1)
a$range <- c("a", "b")würde momentan noch funktionieren weil wir keine checks eingebaut haben. Würde man das am besten mit active bindings umsetzen, oder was meinst du?
Ansonsten würd ich das vielleicht schon mal mergen und den Punkt als neue issue aufnehmen?
Finished the Hyperparameter section