Skip to content

Commit

Permalink
CLN: remove unused interaction_only constructor argument (#537)
Browse files Browse the repository at this point in the history
From PDELibrary and WeakPDELibrary
  • Loading branch information
himkwtn authored Jul 29, 2024
1 parent b3d0b1a commit 90464db
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 18 deletions.
9 changes: 0 additions & 9 deletions pysindy/feature_library/pde_library.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,13 +33,6 @@ class PDELibrary(BaseFeatureLibrary):
temporal_grid : np.ndarray, optional (default None)
The temporal grid if using SINDy-PI with PDEs.
interaction_only : boolean, optional (default True)
Whether to omit self-interaction terms.
If True, function evaulations of the form :math:`f(x,x)` and
:math:`f(x,y,x)` will be omitted, but those of the form :math:`f(x,y)`
and :math:`f(x,y,z)` will be included.
If False, all combinations will be included.
include_bias : boolean, optional (default False)
If True (default), then include a bias column, the feature in which
all polynomial powers are zero (i.e. a column of ones - acts as an
Expand Down Expand Up @@ -90,7 +83,6 @@ def __init__(
derivative_order=0,
spatial_grid=None,
temporal_grid=None,
interaction_only=True,
include_bias=False,
include_interaction=True,
implicit_terms=False,
Expand All @@ -102,7 +94,6 @@ def __init__(
):
self.function_library = function_library
self.derivative_order = derivative_order
self.interaction_only = interaction_only
self.implicit_terms = implicit_terms
self.include_bias = include_bias
self.include_interaction = include_interaction
Expand Down
9 changes: 0 additions & 9 deletions pysindy/feature_library/weak_pde_library.py
Original file line number Diff line number Diff line change
Expand Up @@ -53,13 +53,6 @@ class WeakPDELibrary(BaseFeatureLibrary):
at least one dimension corresponding to a temporal grid, so that
integration by parts can be done in the weak formulation.
interaction_only : boolean, optional (default True)
Whether to omit self-interaction terms.
If True, function evaulations of the form :math:`f(x,x)`
and :math:`f(x,y,x)` will be omitted, but those of the form
:math:`f(x,y)` and :math:`f(x,y,z)` will be included.
If False, all combinations will be included.
include_bias : boolean, optional (default False)
If True (default), then include a bias column, the feature in which
all polynomial powers are zero (i.e. a column of ones - acts as an
Expand Down Expand Up @@ -140,7 +133,6 @@ def __init__(
function_library: Optional[BaseFeatureLibrary] = None,
derivative_order=0,
spatiotemporal_grid=None,
interaction_only=True,
include_bias=False,
include_interaction=True,
K=100,
Expand All @@ -156,7 +148,6 @@ def __init__(
):
self.function_library = function_library
self.derivative_order = derivative_order
self.interaction_only = interaction_only
self.implicit_terms = implicit_terms
self.include_bias = include_bias
self.include_interaction = include_interaction
Expand Down

0 comments on commit 90464db

Please sign in to comment.