Skip to content

dwtools - Data Warehouse tools - #803

Closed
jangorecki wants to merge 6 commits into
Rdatatable:masterfrom
jangorecki:dwtools
Closed

jangorecki wants to merge 6 commits into
Rdatatable:masterfrom
jangorecki:dwtools

Conversation

@jangorecki

Copy link
Copy Markdown
Member

I'm pushing a dwtools branch to reach broader audience, maybe collect some feedback/contribution. I don't want to push it into master without feedback and wider tests.
The functions in the branch are a bit related to Data Warehouse, yet DW is about data processing so they can surly be applied to many other use cases.
You can install it by

# this will work when branch will be accepted
#install_github("data.table", "Rdatatable", build_vignettes=FALSE, ref = "dwtools")
# this points to branch in my repo, should works all the time
install_github("data.table", "jangorecki", build_vignettes=FALSE, ref = "dwtools")

Currently included in the branch are:

In case of contribution please keep the code separated from the master branch - do not write to data.table master files, including NAMESPACE - unless of course you are Rdatatable team and you want to merge something to master. It will make this branch less problematic in terms of merging/pulling.
Also please do not comment here about particular functions, keep discussion focused on the branch. In case of any question related to function you can create a issue and notify me using @jangorecki.

@jangorecki

Copy link
Copy Markdown
Member Author

Wrong target branch. It seems I cannot pull request my branch dwtools to new branch dwtools in your data.table repo. Most likely the branch dwtools must be first created by you.
So, @mattdowle, @arunsrinivasan can you take a look at https://github.com/jangorecki/data.table/tree/dwtools install_github("data.table", "jangorecki", build_vignettes=FALSE, ref = "dwtools")
and fork it as a dwtools branch in data.table repo, or create empty dwtools branch so I can pull request to it.
Thanks

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.

1 participant