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

Update adaptee.py #11

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

JoaoPedroNC
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@leosauberman leosauberman left a comment

Choose a reason for hiding this comment

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

Alguns links de referência:

ParameterGrid no Scikit Learn
GridSearchCV

Os tipos de cada paâmetro você pode encontrar no texto (TCC) ou me perguntar

src/adaptee.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@leosauberman leosauberman left a comment

Choose a reason for hiding this comment

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

João, os meus comentários servem para todo o código. Por favor tire um tempo para estudar sobre Python Docstrings e as guidelines do scikit learn.

Comment on lines +32 to +42
parameters: <<DIZER QUAIS SÃO OS TIPOS ACEITOS>>
<<DEFINIR O QUE É ESSE PARÂMETRO (EXEMPLO ABAIXO)>>
The parameter grid to explore, as a dictionary mapping estimator
parameters to sequences of allowed values.

An empty dict signifies default parameters.

A sequence of dicts signifies a sequence of grids to search, and is
useful to avoid exploring parameter combinations that make no sense
or have no effect. See the examples below.
"""
Copy link
Collaborator

Choose a reason for hiding this comment

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

Precisa alterar essa parte para conter os parametros da classe nesta formatação

src/adaptee.py Outdated
Comment on lines 50 to 55

#Definicao da decodificação, o qual usa como parametro o self, o cromossomo base da variável cromossomo e a variável booleana de reescrita
#Isso converterá seu valor para float
#Ele também retornará o score do self, usando como parâmetro o encoder do self o qual por sua vez o cromossomo como parâmetro
def decode(self, chromosome: BaseChromosome, rewrite: bool) -> float:
return self.score(self.encoder(chromosome))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Isso deveria estar assim:

Suggested change
#Definicao da decodificação, o qual usa como parametro o self, o cromossomo base da variável cromossomo e a variável booleana de reescrita
#Isso converterá seu valor para float
#Ele também retornará o score do self, usando como parâmetro o encoder do self o qual por sua vez o cromossomo como parâmetro
def decode(self, chromosome: BaseChromosome, rewrite: bool) -> float:
return self.score(self.encoder(chromosome))
def decode(self, chromosome: BaseChromosome, rewrite: bool) -> float:
"""
Definicao da decodificação, o qual usa como parametro o cromossomo base da variável cromossomo e a variável booleana de reescrita. O método retorna o resultado da função de avaliação para um dado cromossomo.
Parameters
----------
chromosome: BaseChromosome
<<DEFINIR O QUE É ESSE PARÂMETRO>>
rewrite: bool
<<DEFINIR O QUE É ESSE PARÂMETRO>>
Returns
-------
score: float
resultado da função de avaliação para o cromossomo
"""
return self.score(self.encoder(chromosome))

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