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

Standard optimization test functions #7

Merged
merged 12 commits into from
Jan 30, 2020
Merged

Standard optimization test functions #7

merged 12 commits into from
Jan 30, 2020

Conversation

jairodelgado
Copy link
Collaborator

@jairodelgado jairodelgado commented Oct 2, 2019

Fix #6

@alejandrom247
Copy link
Collaborator

This pull request requires a review a intend to deal with issue #6

Copy link
Collaborator Author

@jairodelgado jairodelgado left a comment

Choose a reason for hiding this comment

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

@alejandrom247 what do you think if we wrap all benchmark functions in a new namespace within solution namespace, e.g.: dnn_opt::core::solutions::benchmark. That way I would suggest to create a new directory inside src/core/solutions and include not only the new benchmark functions you are impementing now but the old ones...

src/core/solutions/brown_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/brown_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/brown_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/brown_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/brown_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds_function.h Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds_function.h Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds_function.h Outdated Show resolved Hide resolved
src/core/solutions/cosine_mixture.cpp Outdated Show resolved Hide resolved
Copy link
Collaborator

@alejandrom247 alejandrom247 left a comment

Choose a reason for hiding this comment

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

This the Csendes plus the standard code revision

Copy link
Collaborator Author

@jairodelgado jairodelgado left a comment

Choose a reason for hiding this comment

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

Coding standards compliance still needs to be improved.

src/core/solutions/brown_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/brown_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/brown_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds_function.h Outdated Show resolved Hide resolved
src/core/solutions/cosine_mixture.cpp Outdated Show resolved Hide resolved
src/core/solutions/cosine_mixture.cpp Outdated Show resolved Hide resolved
src/core/solutions/csendes.cpp Outdated Show resolved Hide resolved
src/core/solutions/brown_function.h Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds.cpp Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds.h Outdated Show resolved Hide resolved
src/core/solutions/cosine_mixture.cpp Outdated Show resolved Hide resolved
src/core/solutions/cosine_mixture.cpp Outdated Show resolved Hide resolved
src/core/solutions/csendes.cpp Outdated Show resolved Hide resolved
@jairodelgado
Copy link
Collaborator Author

I have a dude. The int to reference the position of the array of params can be int or also should be convert in float??

It has to be int. Indexes are integer.

Use float literals only inside aritmetic operations that are meant to be float.

src/core/solutions/brown_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds_function.cpp Outdated Show resolved Hide resolved
src/core/solutions/chung_reynolds_function.h Outdated Show resolved Hide resolved
src/core/solutions/dixonp.cpp Outdated Show resolved Hide resolved
src/core/solutions/eggh.cpp Outdated Show resolved Hide resolved
src/core/solutions/expo.h Outdated Show resolved Hide resolved
src/core/solutions/eggh.h Outdated Show resolved Hide resolved
src/core/solutions/giunta.h Outdated Show resolved Hide resolved
@jairodelgado jairodelgado changed the title Sequential test functions Standard optimization test functions Jan 26, 2020
@jairodelgado jairodelgado merged commit 801f8be into master Jan 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Standard optimization test functions
2 participants