Precise text file parsing (#4081)* New build option: USE_PRECISE_TEXT_PARSER.
Use fast_double_parser for text file parsing. For each number, fallback
to strtod in case of parse failure.
* Add benchmark for CSVParser with Atof and AtofPrecise.
* Fix lint complaint.
* Fix typo in open result error message.
* Revert "Fix lint complaint."
This reverts commit 92ab0b6bce9f17d7be9eaeb20f19d4a0a36f0387.
* Revert "Add benchmark ...
[R-package] Use R standard routines to access character data in C++ (#4252)* converted LGBM_BoosterSaveModel_R
* switch all other non-params cases
* handle params
Correct spelling (#4250)* Correct spelling
Most changes were in comments, and there were a few changes to literals for log output.
There were no changes to variable names, function names, IDs, or functionality.
* Clarify a phrase in a comment
Co-authored-by: James Lamb <jaylamb20@gmail.com>
* Clarify a phrase in a comment
Co-authored-by: James Lamb <jaylamb20@gmail.com>
* Clarify a phrase in a comment
Co-autho...
[R-package] Use R standard routines to access numeric and integer array data in C++ (#4247)* real pointer for matrix
* remove R_REAL_PTR
* remove R_INT_PTR
* add test
[R-package] use R standard routine to access read-only ints passed to C++ (#4246)* [R-package] replace R_AS_INT with R built-in
* update header
* more changes
* Apply suggestions from code review
Co-authored-by: Nikita Titov <nekit94-08@mail.ru>
Co-authored-by: Nikita Titov <nekit94-08@mail.ru>
[R-package] prevent symbol lookup conflicts (fixes #4045) (#4155)* [R-package] prevent symbol lookup conflicts
* add unit tests
* only run test on Windows
* move to .Call() calls
* fix references
* testing registration for CMake builds
* revert NAMESPACE changes
* revert testing changes
[R-package] Move error handling into C++ side (#4163)* [R-package] raise errors from C++ side
* working but a lot of warnings
* more changes
* simplify
* cleanup
* linting
* fix valgrind issues
* revert unnecessary change
[dask] Fix typo mentioned in 4101 (#4214)* fix typo in dask _train as mentioned in 4101
* Update python-package/lightgbm/dask.py
Co-authored-by: James Lamb <jaylamb20@gmail.com>
[ci] parallelize R package installs in CI jobs (#4198)* [ci] parallelize R package installs in CI jobs
* build 2
* build 3
* add ncpus in static_analysis
[R-package] change default nrounds to 100 to match LightGBM core library default (#4197)* use same default arguments as python
* Update R-package/R/lgb.cv.R
Changed.
Co-authored-by: James Lamb <jaylamb20@gmail.com>
* revert change in docs
Co-authored-by: James Lamb <jaylamb20@gmail.com>
[docs][R-package] Update the explanation of num_threads (fixes #4192) (#4199)* update the explanation of num_threads in lightgbm.R
* adjusted sentences for linting.
* remove white space for lintr pass
* add note about num_thread param in files
[R-package] Make returned feature importances from lgb.importance() visible by default (#4194)* Return feature importances as visible
Feature importances are returned as invisible. I don't get why would anyone want them invisible. This PR fixes it.
* Update R-package/R/lgb.importance.R
Thanks, didn't realize that, changed now.
Co-authored-by: James Lamb <jaylamb20@gmail.com>
Co-authored-by: James Lamb <jaylamb20@gmail.com>
[ci] run cpp tests at CI (#4166)* run cpp tests at CI
* Update docs/Installation-Guide.rst
Co-authored-by: James Lamb <jaylamb20@gmail.com>
Co-authored-by: James Lamb <jaylamb20@gmail.com>
[ci] Bump version for development (#4171)* Bump version for development
* Auto-update configure
Co-authored-by: GitHub Actions Bot <githubactionsbot@users.noreply.github.com>
Migrated to f-strings for .\.nuget\create_nuget.py #4136 (#4162)* Migrated to f-strings for .\.nuget\create_nuget.py #4136
* Migrate to f-strings for the Python code for docs/conf.py #4136
* add missing whitespace at ./docs/conf.py:79:14
* Revert "add missing whitespace at ./docs/conf.py:79:14"
This reverts commit 60200849b1a2bd911cc426014ecf1d6e319861b4.
* Revert "Migrate to f-strings for the Python code for docs/conf.py #4136"
This reverts commit ...
[ci] Add debian-clang-devel CI job for the R package (#4164)* [ci] Add debian-clang-devel CI job for the R package
* [ci] Add debian-clang-devel CI job for the R package
* fixes
* fix test
* update cran-comments
* re-trigger CI
* catch warnings and notes
* fail with only a NOTE
* uncomment CI code
* remove env vars in windows tests
* Update R-package/cran-comments.md
Co-authored-by: Nikita Titov <nekit94-08@mail.ru>
Co-authored-by: StrikerRU...
[python-package] Add type hints to the callback file (#4093)* added type hints; implemented one workaround
* resolving some linting errors
* Added doc strings
* fixed more linting errors
* Made documentation more imperative.
* removed one type hint
* more specific type hinting
Co-authored-by: James Lamb <jaylamb20@gmail.com>
* added import
* Apply suggestions from code review
Co-authored-by: James Lamb <jaylamb20@gmail.com>
* made a class and...
Chen Yufei
