Skip to content

Commit

Permalink
add commented sections, vignette
Browse files Browse the repository at this point in the history
  • Loading branch information
tdhock committed Apr 27, 2023
1 parent 80c36ab commit 009d407
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,17 +1,26 @@
#https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners
* @mattdowle

# melt.
/R/fmelt.R @tdhock
/src/fmelt.c @tdhock
/man/melt.data.table.Rd @tdhock
/vignettes/datatable-reshape.Rmd @tdhock

# rolling joins.
/R/froll.R @jangorecki
/man/froll.Rd @jangorecki
/src/froll.c @jangorecki
/src/frollR.c @jangorecki
/src/frolladaptive.c @jangorecki

# meta-programming.
/R/programming.R @jangorecki
/man/substitute2.Rd @jangorecki
/src/programming.c @jangorecki
/vignettes/datatable-programming.Rmd @jangorecki

# time series.
/src/gsumm.c @ben-schwen
/R/IDateTime.R @ben-schwen
/src/idatetime.c @ben-schwen
Expand Down

2 comments on commit 009d407

@jangorecki
Copy link
Member

@jangorecki jangorecki commented on 009d407 Apr 28, 2023

Choose a reason for hiding this comment

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

Comments in this commit are inaccurate

Not rolling joins, but rolling statistics.

Not time series. Shift coule be considered one, but it fits more into "ordered observations". Other is date/time classes. Gsum is gforce grouping.

@tdhock
Copy link
Member Author

@tdhock tdhock commented on 009d407 May 1, 2023

Choose a reason for hiding this comment

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

sorry about that, please correct!

Please sign in to comment.