-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PEcAn 2.0 list of non-backward compatible changes #392
Comments
A few BETY-related changes I'd like to see, while they're fresh on my mind:
|
|
This issue is stale because it has been open 365 days with no activity. |
A suggestion: make it more clear from function names which functions return objects and which are called for their side-effects. E.g. |
Package names
all suggestions can be discussed in comments; checked when finalized
.
's that break Rcpp Period in package name (PEcAn.rtm) conflicts with Rcpp compilation #391Function names
[generic].
s that falsely imply S3 methods, e.g.plot.variance.decomposition
(Stop roxygen from assuming S3 methods #380)Other Conventions
Variable names
reorganization
The text was updated successfully, but these errors were encountered: