Skip to content

Conversation

@Danfoa
Copy link
Collaborator

@Danfoa Danfoa commented Sep 26, 2024

This commit removes the multiple imports to the configuration file from different modules within the pipeline.

This hinders the hability to customize the system's configuration, since each submodule is reloading a fixed / outdated configuration

This commit removes the multiple imports to the configuration file from different modules within the pipeline.

This hinders the hability to customize the system's configuration, since each submodule is reloading a fixed / outdated configuration
@giulioturrisi
Copy link
Collaborator

Hi! This is much needed, i know... I think that many other import are hidden inside the gradient and sampling mpcs, so this need still some rework

@Danfoa
Copy link
Collaborator Author

Danfoa commented Sep 27, 2024

I checked and removed all cfg imports on the project and outside of the simulation script.

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.

2 participants