Resolve "(Learner1D) add possibility to use the direct neighbors in the loss" (!131) · Merge requests · Quantum Tinkerer / adaptive

Merged Jorn Hoofwijk requested to merge 119-add-second-order-loss-to-adaptive into master 6 years ago

Currently works for one

R^1 \to R^1

Still have to make it work for

R^1 \to R^N

Also performance is actually quite good. As in: the learner slows down about 1.5 times. Going (on my laptop) from 3 seconds per 1000 points to 4.5 seconds per 1000 points.

Which, I believe, will be more than compensated by the fact that the chosen points are generally better

Nov 22, 2018
- change loss function signature · fde1774b
  Jorn Hoofwijk authored 6 years ago and Bas Nijholt committed 6 years ago
  
  fde1774b
- fix broken tests and use the new API · 27331280
  Jorn Hoofwijk authored 6 years ago and Bas Nijholt committed 6 years ago
  
  27331280
- added curvature docs · 81a38246
  Jorn Hoofwijk authored 6 years ago and Bas Nijholt committed 6 years ago
  
  81a38246
- introduce 'uses_nth_neighbors' decorator and rename to 'nth_neighbors' · f68bd816
  Bas Nijholt authored 6 years ago
  
  This abstracts the attribute 'nn_neighbors' away and makes it easier for the user, because one can now just set the 'loss_per_interval' and the 'nn_neighbors' will be set be default.
  f68bd816
- introduce 'fast_det' for 2x2 and 3x3 matrices · ae6af683
  Bas Nijholt authored 6 years ago
  
  ae6af683
- rename 'function_values' to 'data' because its more obvious · 49d67d7f
  Bas Nijholt authored 6 years ago
  
  Because data is now in the 'BaseLearner'
  49d67d7f
- remove 'loss_depends_on_neighbours' and replace by 'nn_neighbors' · 88162870
  Bas Nijholt authored 6 years ago
  
  This makes the code work with any number of neighbors. Now the new triangle loss even works with nn_neigbors=0. I also added that we pass the neighbors in all loss_per_interval functions.
  88162870
- added a curvature_loss function to learner1D · 0360e795
  Jorn Hoofwijk authored 6 years ago and Bas Nijholt committed 6 years ago
  
  0360e795

Resolve "(Learner1D) add possibility to use the direct neighbors in the loss"

Merge request reports

Activity