You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
(Issue split off from #33) The stitch() function doesn't currently support chunk options, just chunk names. @yihui requested that I open an issue to serve as a reminder.
I wasn't necessarily advocating that stitch() implement chunk options. Let me play the devil's advocate for a moment. It may be preferable to keep things as you have them, and avoid adding complexity to a derivative function like stitch() because,
a simpler function may be easier for less experienced users to start with.
its additional functionality is one more thing you have to maintain and regression test.
As long as the documentation redirects the user to the appropriate function, I don't have a strong opinion either way.
The text was updated successfully, but these errors were encountered:
(Issue split off from #33)
The
stitch()
function doesn't currently support chunk options, just chunk names. @yihui requested that I open an issue to serve as a reminder.I wasn't necessarily advocating that
stitch()
implement chunk options. Let me play the devil's advocate for a moment. It may be preferable to keep things as you have them, and avoid adding complexity to a derivative function likestitch()
because,As long as the documentation redirects the user to the appropriate function, I don't have a strong opinion either way.
The text was updated successfully, but these errors were encountered: