Refactoring of charges#10
Conversation
|
Some remarks:
|
|
Ok, first sorry for the delay; I dont' really have time to maintain this project (which was originally a toy project :) ) . The encoding/decoding was originally thought to be the first step toward support for image recognition of blason: if you can represent a blason as a vector, you can train a neural network that way. I never had time to work on that, so it can be trashed, I guess. As this is a toy project, I set some restriction to myself, like "don't use classes" (I'm a functionnal programmer lover). As it seems your don't really follow this philosophy, it's all the more a reason for you to work on your fork (instead of fighting each other on petty things like "is inheritance a bad thing?"). Sorry to not be more available at the moment :( |
This change is meant to facilitate later additions of charges by regrouping all data relative to a particular charge inside a dedicated class that can be inherited.