-
Notifications
You must be signed in to change notification settings - Fork 67
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
Code hierarchy cleanup #137
Labels
Comments
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now the code dependency graph is pretty complex:
(You can see instructions on the README for generating this)
I think this complexity can be greatly reduced, with the dependencies made more hierarchical.
Core.jl
is one such example.For example, I think the following files can be grouped, and made into combined modules:
EquationUtils.jl
,EvaluateEquation.jl
,EvaluateEquationDerivative.jl
,SimplifyEquation.jl
, andInterfaceSymbolicUtils.jl
HallOfFame.jl
,Population.jl
, andPopMember.jl
Mutate.jl
andMutationFunctions.jl
RegularizedEvolution.jl
andSingleIteration.jl
The tricky part will be ensuring that there is still a clear hierarchy. I think the above order is correct, in that
i
should not depend oni+1
, but we might need to move functions around so that the dependency structure is still satisfied.If anybody is interested in working on this, let me know.
The text was updated successfully, but these errors were encountered: