Skip to content

Conversation

sl-solution
Copy link

InMemoryDatasets.jl is a new package for data manipulation in Julia. Please let me know if anything is wrong.

InMemoryDatasets.jl is a new package for data manipulation in Julia.
Copy link
Contributor

@jangorecki jangorecki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR looks very good. Just to let you know that I am no longer maintaining the project (although I am happy to provide the feedback) and the best way to have your PR merged and run is, I believe, contacting h2o.ai support. Project is inactive for a longer while but I haven't heard anything about it to be abandoned. Looking forward to see benchmark results.

@@ -0,0 +1,58 @@
using Printf; # sprintf macro to print in non-scientific format
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAIU this file could be shared between juliadf and juliads. As we do for pyton and R tools.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMD uses DLMReader for writing the log file, however, DataFrames.jl uses CSV. So one possible way to combine these is to have different write_log functions in the same file.

using DLMReader;
using PooledArrays

IMD.warmup()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice to have a comment what this warm up actually do

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a comment for this.

@sl-solution
Copy link
Author

Thanks for your feedback. I have made some changes based on your comments.
Hope to see the project becomes active again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants