Skip to content

Conversation

@benholmgren
Copy link
Contributor

Roxygen style comments for parameters and return value, comments in code

Copy link
Contributor

@dlm dlm left a comment

Choose a reason for hiding this comment

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

I added a few more suggestions. Feel free to address using the feedback on the previous PR. We can discuss further on Mon too and then merge in.


# with weight
} else {
# establish the weightbound and weight parameters to be used in final DtmWeight function
Copy link
Contributor

Choose a reason for hiding this comment

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

These and the next set are great comments. The code below is somewhat non-obvious and the comments make it very easy to read what is happening.

Streamlined comments again in similar manner to previous PR
roxygen needs a title to make nice documentation reliably
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