Skip to content

Commit

Permalink
Remove option for replica seeds to be None in Preprocessing
Browse files Browse the repository at this point in the history
  • Loading branch information
APJansen committed Jan 29, 2024
1 parent f083440 commit 8adc0ae
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions n3fit/src/n3fit/layers/preprocessing.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@ class Preprocessing(MetaLayer):
Parameters
----------
replica_seeds: List[int]
list of pre replica seeds for the initializer of the random alpha and beta values
flav_info: list
list of dicts containing the information about the fitting of the preprocessing factor
This corresponds to the `fitting::basis` parameter in the nnpdf runcard.
Expand All @@ -31,16 +33,14 @@ class Preprocessing(MetaLayer):
(defaults to true)
large_x: bool
Whether large x preprocessing factor should be active
replica_seeds: List[int]
list of pre replica seeds for the initializer of the random alpha and beta values
num_replicas: int (default 1)
The number of replicas
"""

def __init__(
self,
replica_seeds: Optional[List[int]],
flav_info: Optional[list] = None,
replica_seeds: Optional[List[int]] = None,
large_x: bool = True,
num_replicas: int = 1,
**kwargs,
Expand All @@ -50,10 +50,7 @@ def __init__(
"Trying to instantiate a preprocessing factor with no basis information"
)
self.flav_info = flav_info
if replica_seeds is None:
self.replica_seeds = [0] * num_replicas # TODO Aron: figure out if this ever happens
else:
self.replica_seeds = replica_seeds
self.replica_seeds = replica_seeds
self.large_x = large_x
self.num_replicas = num_replicas

Expand Down

0 comments on commit 8adc0ae

Please sign in to comment.