-
Notifications
You must be signed in to change notification settings - Fork 278
Open
Labels
Description
Dear all,
With latest update, which allowed to increase the number of maximum nodes using kwargs dict inside registry(...) function, the default parameters are 100 nodes,
| maximum_node_count: int = 100, |
Although, that does not change the logic of computing
action_mask and set valid node id to be nax number of discovered nodes, internally vector is set to be too big.
- Question: I want to ask for the logic of change from using
max_node_countparameter here, why we want to have option to enlarge the maximum nodes count using registry and this we had to change the value from inferred form initial_network to the one included in bounds for registry function?max_node_count = self.bounds.maximum_node_count - Solution: The workaround as I see can be simply to make this parameters
maximum_node_count: int | Noneand then make if else logic for setting them equal to initial number of parameters from network (done in previous commits from main)