Skip to content

Commit 42095f7

Browse files
committed
fixup from rebase
1 parent 7a3e0ce commit 42095f7

2 files changed

Lines changed: 2 additions & 12 deletions

File tree

petab/v2/lint.py

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1011,20 +1011,9 @@ def get_valid_parameters_for_parameter_table(
10111011
if p not in invalid
10121012
)
10131013

1014-
# Add petab ids from mapping table if they are used for aliasing
1015-
# FIXME only add mapping.petab_id to allowed parameter IDs list if it
1016-
# aliases an invalid PEtab ID? See
1017-
# https://github.com/PEtab-dev/libpetab-python/pull/482#discussion_r3420762034
10181014
for mapping in problem.mappings:
1019-
if mapping.petab_id not in invalid:
1015+
if mapping.model_id and mapping.model_id in parameter_ids.keys():
10201016
parameter_ids[mapping.petab_id] = None
1021-
# An aliased model id is not a valid parameter id
1022-
if (
1023-
mapping.model_id
1024-
and mapping.model_id != mapping.petab_id
1025-
and mapping.model_id in parameter_ids
1026-
):
1027-
del parameter_ids[mapping.model_id]
10281017

10291018
# add output parameters from observable table
10301019
output_parameters = problem.get_output_parameters()

tests/v2/test_lint.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77

88
from petab.v2 import Problem
99
from petab.v2.lint import *
10+
from petab.v2.models.pysb_model import PySBModel
1011
from petab.v2.models.sbml_model import SbmlModel
1112

1213

0 commit comments

Comments
 (0)