-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Closed
Labels
APIIssues pertaining the friendly APIIssues pertaining the friendly API
Milestone
Description
The trained Generalized Additive Models (aka GAMs) methods for obtaining trained model parameters have non-standard names and various inconsistencies.
These are the properties:
var shapeFunctionsBins = gamModel.GetBinUpperBounds();
var shapeFunctionsValues = gamModel.GetBinEffects();
var numShapeFunctions = gamModel.NumShapeFunctions;
var intercept = gamModel.Intercept;There are a few issues here:
- Nonstandard names
I proposeBinUpperBoundsandBinEffectsbe renamedShapeFunctionsBinsandShapeFunctionsBinValues. - Inconsistent names
Sometimes we sayShapeFunctions, sometimes we done. Same solution proposed as No. 1. - Inconsistency in
Getmethods vs. properties
e.g.Interceptvs.GetBinUpperBounds.
I propose we move all the methods to be properties.
e.g.model.ShapeFunctionsBins. - Names leak implementation
BinUpperBoundsvs.Bins.
Question: should we just sayBinsrather thanBinUpperBoundsand say that they are the upper bounds for the bin in the documentation?BinUpperBoundsis very verbose, and speaks to how we bin.
Metadata
Metadata
Assignees
Labels
APIIssues pertaining the friendly APIIssues pertaining the friendly API