Jay Qi

Right now, test coverage in `FunctionReporter` does not work with R6 methods. Presumably if Reference Class methods were supported (#197), they would not work either. All of those turn up...

Per discussion in #183 and #181, as well as the proposed change in #198 to unify documentation around the terminology "network reporters", I think we should consider renaming `AbstractGraphReporter` to...

Currently the graph reporters only have the table of nodes and node measures in their summary view. It would be useful to also include the network measures.

Logging an issue where `FunctionReporter` is having trouble parsing some functions, with the suspected cause being related to the use of decorators. Thank you to @Manusreekumar who reported this offline...

Discussion: separate set_package and set_package_path methods

1

While reviewing #129, @jameslamb had the idea: > ok next idea related to our ongoing discussion today re #106 .... > > I think we should make reporters have a...

There are no instructions or guidelines currently for either technical or content contributions.

Establish consistent capitalization

1

There are inconsistent stylings of **deon** lowercase and **Deon** uppercase throughout the documentation. There should be an agreed-upon styling, and the other way should be fixed. As for considerations: -...

Normally, when you try to define public and private members in the same class with colliding names, you get an error. ``` r MyClass Error in R6::R6Class("MyClass", public = list(colliding_method...

Add examples in roxygen comments to tests Example: https://github.com/terrytangyuan/dml/blob/master/R/gdmf.r#L42

Travis tests for different os Example: https://github.com/dmlc/xgboost/blob/master/.travis.yml#L26