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

Some concerns about our neural networks and Ctrnn #35

Open
Ploppz opened this issue Feb 24, 2019 · 0 comments
Open

Some concerns about our neural networks and Ctrnn #35

Ploppz opened this issue Feb 24, 2019 · 0 comments

Comments

@Ploppz
Copy link

Ploppz commented Feb 24, 2019

Organism and Gene

  • In mutation_add_connection, there is no check whether the connection already exists. In Organism::get_weights, we only use the weight of one connection, so any other similar connections are useless.
  • Bias can only be positive. I'm not sure if this is good? I also noticed in Ctrnn that we take the negative bias, rather than the positive as in the paper. So practically we always have a negative bias.
  • What about having both bias: f64 and weight: f64 in Gene (instead of having a bias and a weight type of gene)?

Ctrnn

  • Shouldn't tau be bigger than 1.0? I think it sort of represents how fast y decays. And when it is 1.0, y decays in only one timestep. (you can verify this by looking at how we calculate the new y (however look at my commit where I fixed a small thing in Ctrnn)
@Ploppz Ploppz changed the title Some concerns about Ctrnn Some concerns about our neural networks and Ctrnn Feb 24, 2019
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

No branches or pull requests

1 participant