Skip to content

Commit

Permalink
Remove unused kwargs from Model constructor
Browse files Browse the repository at this point in the history
Closes #4981
  • Loading branch information
itsguneetsingh authored Sep 23, 2021
1 parent 8e54fc9 commit b61daa1
Showing 1 changed file with 1 addition and 13 deletions.
14 changes: 1 addition & 13 deletions pymc3/model.py
Original file line number Diff line number Diff line change
Expand Up @@ -518,16 +518,6 @@ class Model(Factor, WithMemoization, metaclass=ContextMeta):
name: str
name that will be used as prefix for names of all random
variables defined within model
model: Model
instance of Model that is supposed to be a parent for the new
instance. If ``None``, context will be used. All variables
defined within instance will be passed to the parent instance.
So that 'nested' model contributes to the variables and
likelihood factors of parent model.
aesara_config: dict
A dictionary of Aesara config values that should be set
temporarily in the model context. See the documentation
of Aesara for a complete list.
check_bounds: bool
Ensure that input parameters to distributions are in a valid
range. If your model is built in a way where you know your
Expand All @@ -551,7 +541,7 @@ class Model(Factor, WithMemoization, metaclass=ContextMeta):
class CustomModel(Model):
# 1) override init
def __init__(self, mean=0, sigma=1, name='', model=None):
def __init__(self, mean=0, sigma=1, name=''):
# 2) call super's init first, passing model and name
# to it name will be prefix for all variables here if
# no name specified for model there will be no prefix
Expand Down Expand Up @@ -627,8 +617,6 @@ def __new__(cls, *args, **kwargs):
def __init__(
self,
name="",
model=None,
aesara_config=None,
coords=None,
check_bounds=True,
rng_seeder: Optional[Union[int, np.random.RandomState]] = None,
Expand Down

0 comments on commit b61daa1

Please sign in to comment.