Skip to content

Commit

Permalink
Problem.add_parameter: Rename kwarg estimated->estimate
Browse files Browse the repository at this point in the history
Change the `estimated` argument in Problem.add_parameter to `estimate`
to match the column name.
  • Loading branch information
dweindl committed Dec 18, 2024
1 parent cc33da2 commit 70d3bfe
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 12 deletions.
10 changes: 4 additions & 6 deletions petab/v1/problem.py
Original file line number Diff line number Diff line change
Expand Up @@ -1074,7 +1074,7 @@ def add_observable(
def add_parameter(
self,
id_: str,
estimated: bool | str | int = True,
estimate: bool | str | int = True,
nominal_value=None,
scale: str = None,
lb: Number = None,
Expand All @@ -1089,7 +1089,7 @@ def add_parameter(
Arguments:
id_: The parameter id
estimated: Whether the parameter is estimated
estimate: Whether the parameter is estimated
nominal_value: The nominal value of the parameter
scale: The parameter scale
lb: The lower bound of the parameter
Expand All @@ -1104,11 +1104,9 @@ def add_parameter(
record = {
PARAMETER_ID: [id_],
}
if estimated is not None:
if estimate is not None:
record[ESTIMATE] = [
int(estimated)
if isinstance(estimated, bool | int)
else estimated
int(estimate) if isinstance(estimate, bool | int) else estimate
]
if nominal_value is not None:
record[NOMINAL_VALUE] = [nominal_value]
Expand Down
10 changes: 4 additions & 6 deletions petab/v2/problem.py
Original file line number Diff line number Diff line change
Expand Up @@ -829,7 +829,7 @@ def add_observable(
def add_parameter(
self,
id_: str,
estimated: bool | str | int = True,
estimate: bool | str | int = True,
nominal_value=None,
scale: str = None,
lb: Number = None,
Expand All @@ -844,7 +844,7 @@ def add_parameter(
Arguments:
id_: The parameter id
estimated: Whether the parameter is estimated
estimate: Whether the parameter is estimated
nominal_value: The nominal value of the parameter
scale: The parameter scale
lb: The lower bound of the parameter
Expand All @@ -859,11 +859,9 @@ def add_parameter(
record = {
PARAMETER_ID: [id_],
}
if estimated is not None:
if estimate is not None:
record[ESTIMATE] = [
int(estimated)
if isinstance(estimated, bool | int)
else estimated
int(estimate) if isinstance(estimate, bool | int) else estimate
]
if nominal_value is not None:
record[NOMINAL_VALUE] = [nominal_value]
Expand Down

0 comments on commit 70d3bfe

Please sign in to comment.