search

HyperCodec / neat

Crate for implementing NeuroEvolution of Augmenting Topologies
MIT License
4 stars 1 forks source link

Useless atomic operations? #4

Closed HyperCodec closed 7 months ago

HyperCodec commented 7 months ago

NeuralNetworkTopology uses Arc<RwLock<NeruonTopology>> but never uses async code. It would be better to only use something like Rc<RefCell<NeuronTopology>>.

HyperCodec commented 7 months ago

Still going to implement these changes, but it will be a little while as I have more important issues to address first (will still be in the next release, might be one of the later-implemented changes though)

HyperCodec commented 7 months ago

I think I added these ops because of the Send requirement, I should probably extract that out into a feature (both on the genetic-rs side and here). I think these changes will have to wait until a future release.