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.

Sign up for GitHub

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

Jump to bottom

unmerged changes + polish #36

Merged
merged 18 commits into from
Feb 24, 2019
Merged

unmerged changes + polish #36

merged 18 commits into from
Feb 24, 2019

Conversation

Ithanil
Copy link
Member

@Ithanil Ithanil commented Feb 24, 2019

  • added exponential activation function
  • transition from autotools to CMake, retaining and often improving nearly all old functionality
  • some bug fixes made before paper production
  • smaller changes and new benchmark added after paper production
  • improved use of OpenMP
  • another significant performance boost to variational derivatives (more to come)

Ithanil added 17 commits October 17, 2018 01:54
@Ithanil
Add exponential activation function
6969a40
@Ithanil
fix tiny bug
f871c0a
@Ithanil
hack to random initialization during training, to make fitting more s…
4777c13 
…table for setups with feature maps
@Ithanil
any (std::)abs() -> fabs(), to be consistent and safe
85129c5
@Ithanil
Small configure.ac change that was used on Oculus cluster
b6e8b25
@Ithanil
Small bug fix in get/setVP methods (that luckily didn't affect any pr…
7ea4bb6 
…oduction runs)
@Ithanil
Transition to cmake, but not perfectly finished. TravisCI, new file s…
1f38da0 
…tructure and comfortable benchmarking/profiling will be done later.
@Ithanil
adapted true include/src split (no symlinks), to match the new style …
bf4f819 
…of other NNVMC repos
@Ithanil
Change directory structure to match the other NNVMC repos
1f91f37
@Ithanil
Made build.sh more portable and fixed OpenMP build
6e51a9e
@Ithanil
adapted travis.yml
21af1aa
@Ithanil
brought convenient benchmark/profiling functionality back
9e0f845
@Ithanil
Unify READMEs and travis
a550e0d
@Ithanil
small changes to benchmarks/tests and added new benchmark concerning …
7b17e34 
…different network sizes
@Ithanil
improved the way OpenMP is handled
91ccf5b
@Ithanil
two simple changes for significant speedup & reduced memory consumpti…
7ff0548 
…on for large networks
@Ithanil
fix cmakelist typo and mini changes to feeders
3069f1a
@Ithanil Ithanil added this to the v0.2 milestone Feb 24, 2019
@Ithanil Ithanil self-assigned this Feb 24, 2019
@coveralls
Copy link

coveralls commented Feb 24, 2019 •
edited
Loading

Pull Request Test Coverage Report for Build 260

  • 70 of 85 (82.35%) changed or added relevant lines in 15 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.3%) to 79.813%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/fmap/EuclideanDistanceMap.cpp 0 1 0.0%
src/fmap/EuclideanPairDistanceMap.cpp 0 1 0.0%
include/ffnn/actf/ExponentialActivationFunction.hpp 2 5 40.0%
src/train/NNTrainer.cpp 1 5 20.0%
src/net/FeedForwardNeuralNetwork.cpp 9 15 60.0%
Totals Coverage Status
Change from base Build 239: -1.3%
Covered Lines: 2214
Relevant Lines: 2774
💛 - Coveralls

@Ithanil Ithanil merged commit 2c03fa4 into master Feb 24, 2019
@Ithanil Ithanil deleted the polish branch February 24, 2019 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Ithanil Ithanil

Labels
None yet
Projects
None yet
Milestone
v0.2
Development

Successfully merging this pull request may close these issues.
2 participants
@Ithanil @coveralls