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
arff deserialiser #4627
Open
gf712
wants to merge
21
commits into
shogun-toolbox:develop
Choose a base branch
from
gf712:arff_format
base: develop
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
arff deserialiser #4627
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
7c705c6
arff deserialiser
gf712 9425e62
added stream constructor
gf712 53f8925
clean up tokenisation
gf712 22343c6
initial datetime support
gf712 14b0d49
use std::get_time from stl
gf712 2b23d96
added date c++ library for datetime parsing
gf712 056583c
fix msvc error
gf712 cc9b8b3
more tests
gf712 06844aa
ignore missing values
gf712 f4cf772
added a couple more tests
gf712 46b9158
added feature name getter
gf712 4c3346d
minor fixes
gf712 82ae47e
much faster parsing!
gf712 8591a16
added option to choose parsing type
gf712 6b542a1
fixed parsing of strings in quotes
gf712 892d911
moved element count check to before loop
gf712 02b5f81
fix msvc bug
gf712 24d1303
typed tests for numeric values
gf712 2ee0718
added more feature getters
gf712 cbf50e0
arff serialiser
gf712 f0e934b
fix bugs
gf712 File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
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,16 @@ | ||
include(ExternalProject) | ||
set(DATE_PREFIX ${CMAKE_BINARY_DIR}/date) | ||
set(DATE_SOURCE_DIR "${THIRD_PARTY_DIR}/date") | ||
set(DATE_INCLUDE_DIR "${DATE_SOURCE_DIR}/include") | ||
ExternalProject_Add( | ||
date | ||
PREFIX ${DATE_PREFIX} | ||
SOURCE_DIR ${DATE_SOURCE_DIR} | ||
GIT_REPOSITORY https://github.com/HowardHinnant/date.git | ||
GIT_TAG e7e1482087f58913b80a20b04d5c58d9d6d90155 | ||
CONFIGURE_COMMAND "" | ||
BUILD_COMMAND "" | ||
INSTALL_COMMAND ${CMAKE_COMMAND} -E copy_directory ${DATE_INCLUDE_DIR}/ ${THIRD_PARTY_INCLUDE_DIR}/ | ||
) | ||
|
||
add_dependencies(libshogun date) |
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
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
Oops, something went wrong.
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.
seems ok to me to have this
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.
yea, this will be part of the stl in C++20, so might as well start using it
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.
hahahaha
Ah man, finally in c++20, c++ will have nice date parsing... things are changing fast these days ;)