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

ListCrossoverPipeline: report second parent only if actually used #79

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ZvikaZ
Copy link

@ZvikaZ ZvikaZ commented May 24, 2021

If using misc to keep parents' indices, it used to always keep the indices of both parents even if if the reproduction failed, and the individual is just a copy of a single parent. Now it always keep the index of the first parent, and adds the index of the second parent only if it's really used.

@ZvikaZ
Copy link
Author

ZvikaZ commented Aug 9, 2021

Hi.
Any updates with this?

@eclab
Copy link
Collaborator

eclab commented Aug 25, 2021

Okay, I'm not convinced this is a good idea. The point of the parents mechanism is to give you access to the parents that produced children during an operation. That is, you have a breeding pipeline operation op(parent1, parent2) -> { children }. Now it may be the case that children has only one child. But you still may want to know what the two parents were in order to understand why the operation did what it did. IIRC, it should generally be the case that the first parent generated the child if there's only one, so it should be clear which parent is which in the one-child situation, right?

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