Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optionally add CPGs to passive modules #571

Closed
wants to merge 5 commits into from
Closed

Conversation

andresgr96
Copy link
Contributor

  1. Added option to add CPGs to passive modules in addition to active ones
  2. CPGs on passive modules are not assigned a target and thus cannot be controlled and do not produce an output
  3. It was discovered that doing so results in better fitness, thus we added the option to do so so users can experiment with it
  4. Example scripts that utilized it were updated

@andresgr96 andresgr96 requested a review from JedMuff September 12, 2024 12:27
@andresgr96 andresgr96 changed the title Optinally add CPGs to passive modules Optionally add CPGs to passive modules Sep 12, 2024
@andresgr96
Copy link
Contributor Author

I've realized that this seemingly small change introduces a fundamental change in the concept of the CPG network we currently use, and thus will require a deeper discussion to add it to the framework.

@andresgr96 andresgr96 closed this Sep 12, 2024
@andresgr96 andresgr96 deleted the passive_cpgs branch September 12, 2024 19:13
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