Skip to content

add layer duplication check#25

Open
Joao-Pedro-MB wants to merge 25 commits into
developfrom
check-layer-duplication
Open

add layer duplication check#25
Joao-Pedro-MB wants to merge 25 commits into
developfrom
check-layer-duplication

Conversation

@Joao-Pedro-MB

Copy link
Copy Markdown
Collaborator

No description provided.

@EltonCN

EltonCN commented Jun 29, 2023

Copy link
Copy Markdown
Owner

Sobre a checagem do tamanho da população, basicamente a ideia é realizar a checagem que ocorre em https://github.com/EltonCN/evolvepy/blob/204a41b860164e554f7b58bcaaa2561ed71f8d0c/src/evolvepy/generator/generator.py#L229C9-L229C9, só que na definição do gerador.

Considere um tamanho arbitrário para a população na entrada da primeira camada, e verifique se o tamanho se mantém (sem realizar as operações). Talvez tenha que alterar algo na classe de Layer para indicar como ela altera o tamanho da população sem executá-la?

O card no Notion está incorreto, já que o descriptor não contém o tamanho da população.

@EltonCN

EltonCN commented Jun 29, 2023

Copy link
Copy Markdown
Owner

A branch target do pull request deveria ser a develop

Comment thread src/evolvepy/generator/generator.py Outdated
Comment thread src/evolvepy/generator/generator.py Outdated
Comment thread src/evolvepy/generator/generator.py Outdated
Comment thread src/evolvepy/generator/generator.py Outdated
layer_names.add(layer.name)

for next_layer in layer.next:
if next_layer in visited:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acredito que para verificar isso é importante perceber que o único tipo de camada que pode estar no fim dos fluxos (C no primeiro e D no segundo) é a Concatenate.

Quando a análise chegar nela, vc não poder contar +1x as camadas depois (só se o fluxo vier de outra parte).

@EltonCN

EltonCN commented Jun 29, 2023

Copy link
Copy Markdown
Owner

Faltam teste para verificar se os erros/exceções são lançados. Também seria importante preparar um com fluxos diferentes e Concatenate para se certificar que não é lançada exceção.

Repository owner deleted a comment from EltonCN Jun 30, 2023
Repository owner deleted a comment from EltonCN Jun 30, 2023
@Joao-Pedro-MB Joao-Pedro-MB changed the base branch from main to develop June 30, 2023 22:40
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.

3 participants