Skip to content

GitLab

Switch branch/tag
  1. 19 Mar, 2021 1 commit
  3. 17 Mar, 2021 1 commit
    • ashok-ponnuswami-msft's avatar
      Range check for DCG position discount lookup (#4069) · 4580393f
      ashok-ponnuswami-msft authored 
      * Add check to prevent out of index lookup in the position discount table. Add debug logging to report number of queries found in the data.

* Change debug logging location so that we can print the data file name as well.

* Revert "Change debug logging location so that we can print the data file name as well."

This reverts commit 3981b34bd6e0530f89c4733e78e6b6603bf50d48.

* Add data file name to debug logging.

* Move log line to a place where it is output even when query IDs are read from a separate file.

* Also add the out-of-range check to rank metrics.

* Perform check after number of queries is initialized.

* Update
      4580393f
  5. 16 Mar, 2021 4 commits
  7. 15 Mar, 2021 4 commits
  9. 14 Mar, 2021 1 commit
  11. 12 Mar, 2021 2 commits
  13. 11 Mar, 2021 2 commits
  15. 10 Mar, 2021 5 commits
  17. 09 Mar, 2021 2 commits
  19. 05 Mar, 2021 1 commit
  21. 04 Mar, 2021 2 commits
  23. 03 Mar, 2021 1 commit
  25. 02 Mar, 2021 2 commits
  27. 24 Feb, 2021 6 commits
  29. 23 Feb, 2021 2 commits
  31. 22 Feb, 2021 1 commit
  33. 21 Feb, 2021 2 commits
    • mjmckp's avatar
      Fix evalution of linear trees with a single leaf. (#3987) · 605c97b5
      mjmckp authored 
      

* Fix index out-of-range exception generated by BaggingHelper on small datasets.

Prior to this change, the line "score_t threshold = tmp_gradients[top_k - 1];" would generate an exception, since tmp_gradients would be empty when the cnt input value to the function is zero.

* Update goss.hpp

* Update goss.hpp

* Add API method LGBM_BoosterPredictForMats which runs prediction on a data set given as of array of pointers to rows (as opposed to existing method LGBM_BoosterPredictForMat which requires data given as contiguous array)

* Fix incorrect upstream merge

* Add link to LightGBM.NET

* Fix indenting to 2 spaces

* Dummy edit to trigger CI

* Dummy edit to trigger CI

* remove duplicate functions from merge

* Fix evalution of linear trees with a single leaf.

Note that trees without linear models at the leaf always handle num_leaves = 1 as a special case and directly output the leaf value.  Linear trees were missing this special case handling, and hence would have the following issues:
 * Calling Tree::Predict or Tree::PredictByMap would cause an access violation exception attempting to access the first value of the empty split_feature_ array in GetLeaf.
 * PredictionFunLinear would either cause an access violation or go into an infinite loop when attempting to do the equivalent of GetLeaf.

Note also that PredictionFun does not need the same changes as PredictionFunLinear, since both are only called by Tree::AddPredictionToScore, which has a special case for (!is_linear_ && num_leaves_ <= 1) that precludes calling PredictionFun.
Co-authored-by: default avatarmatthew-peacock <matthew.peacock@whiteoakam.com>
Co-authored-by: default avatarGuolin Ke <guolin.ke@outlook.com>
      605c97b5
    • James Lamb's avatar
      [ci] prefer older binary to new source for R packages on Mac builds (fixes #4008) (#4010) · b1d382ee
      James Lamb authored 
      * [ci] prefer older binary to new source for R packages

* back to binary

* preserve choice on Linux
      b1d382ee
  35. 20 Feb, 2021 1 commit