Skip to content

Allow configuration of the broker in local environments#74

Open
josephmckinsey wants to merge 7 commits intomainfrom
jm/broker_config
Open

Allow configuration of the broker in local environments#74
josephmckinsey wants to merge 7 commits intomainfrom
jm/broker_config

Conversation

@josephmckinsey
Copy link
Collaborator

@josephmckinsey josephmckinsey commented Feb 2, 2026

Creates a new HELICSFederateConfig which people are intended to subclass from.

  • It has a "apply_to_federate_info" function that lets you set the properties on the FederateInfo HELICS object to use the correct broker, etc.
  • It should be compatible with helics.helicsFederateInfoLoadFromString when you specify the static_inputs.json file path (although this does not set all the required properties most federates need like period information).
  • It only contains name, broker, and communication information and not timing information.
  • There is a .from_multicontainer class which lets you combine params with the existing multicontainer BrokerConfig (which has slightly different names.

The WiringDiagram now supports setting the broker information in shared_helics_config or in components under helics_config_override as a SharedFederateConfig class which includes only the broker information and no names.

The multicontainer does not support setting the broker information with an override right now, since the configurations are slightly different. We may be able to deprecate the multicontainer BrokerConfig at a later date.

@AadilLatif AadilLatif self-requested a review February 2, 2026 23:31
@josephmckinsey
Copy link
Collaborator Author

I just realized that we don't pass the broker configuration to the broker when we make the run script. Will fix tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant