-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
[fast-double-parser] Add the Fast Double Parser port #39421
Merged
BillyONeal
merged 9 commits into
microsoft:master
from
dsmith111:smithdavi/fast_double_parser
Jun 21, 2024
Merged
Changes from all commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
d4f35f8
Fast double port
dsmith111 6663cf2
Clean up comments
dsmith111 327e1e9
Version database
dsmith111 f2387e5
Fix naming; Remove usage
dsmith111 41ebc8c
Version database
dsmith111 92cb362
Add white-space
dsmith111 c931e1e
Version database
dsmith111 34b6e9f
Remove redundant dependencies/commands
dsmith111 4a4c756
Version database
dsmith111 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
vcpkg_from_github( | ||
OUT_SOURCE_PATH SOURCE_PATH | ||
REPO lemire/fast_double_parser | ||
REF "v${VERSION}" | ||
SHA512 14a23b9e2ddc924d66f2748134364c546c6511ad380fbf59313c1d77c14dc4b0d22367a44b423f835ff2941f8d9c511f18b97ce25eda826ddc08a8eaaf7014bd | ||
HEAD_REF master | ||
) | ||
|
||
set(VCPKG_BUILD_TYPE release) # header-only port | ||
|
||
file(INSTALL "${SOURCE_PATH}/include/" DESTINATION "${CURRENT_PACKAGES_DIR}/include") | ||
|
||
vcpkg_install_copyright(FILE_LIST "${SOURCE_PATH}/LICENSE") |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
{ | ||
"name": "fast-double-parser", | ||
"version": "0.7.0", | ||
"description": "Fast function to parse strings into double (binary64) floating-point values, enforces the RFC 7159 (JSON standard) grammar: 4x faster than strtod", | ||
"homepage": "https://github.com/lemire/fast_double_parser", | ||
"license": "Apache-2.0" | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
{ | ||
"versions": [ | ||
{ | ||
"git-tree": "fac99fe686dd7874d02be06d78f4df1a6ac7358d", | ||
"version": "0.7.0", | ||
"port-version": 0 | ||
} | ||
] | ||
} |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One question: 0.7.0 was released in 2022, why not just update to 0.8.0, which is the lastest version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See the mentioned PR. I was asked to devendor the lightgbm port. The LightGBM package is pinned to this version of fast double parser. But, when the LightGBM library is updated to use the v0.8.0 fast double parser library. I'll of course have to update the version here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For ubiquity:
#39395 (comment)
https://github.com/microsoft/LightGBM/tree/master/external_libs