Comments (4)
RE your item 3 above:
I think your idea is a good one. Your database can have a column with an optional message
for each row (equation), and a column with the condition
that throws the error, e.g.
any( DBH < min_DBH, DBH > max_DBH )
The contents of the column condition
should be valid R code; if it evaluetes to TRUE
, then the message is produced. If the issues may vary in severity, we may need an additional column giving the condition_type
such as inform
, warn
, or abort
-- that is, an instruction to tell R what to do appart from producing a message.
from allodb.
Conclusion regarding statement above based on discussion: As we're controlling the package, there shouldn't be any instances that would warrant "abort". All messages entered in @gonzalezeb's tables should be interpreted as warnings. @maurolepore's code will throw some general warnings.
from allodb.
Regarding [2] in the top comment, this sort of error will not be possible within the master version. However, users could edit the tables offline and run the code. For this reason, it may make sense for the code to include an abort function to deal with unacceptable practices (e.g., if user attempts to apply an allometry way outside its range).
from allodb.
@cpiponiot Let's try to discuss some aspects of this issue today 5/12.
from allodb.
Related Issues (20)
- Replace data.frame() with tibble()
- When calling stop() use call. = FALSE
- some commits not showing
- Increase test coverage
- Tidy allodb::equations in advance, not in new_equaions()
- Prune data-raw/ HOT 1
- Remember to review FIXME and TODO
- Simplify new_equations()
- Ensure error messages wrap nicely
- Remove needless calls to print() in examples
- Use simpler markdown syntax, e.g. replace \pkg{allodb} with *allodb*
- Review punctuation of helpfiles (title and other sections)
- Use simple comments to follow tidyverse styleguide "# blah" not "## blah"
- Reserve return() for early returns (tidyverse styleguide)
- Remove redundant reference to allodb
- Remove duplication in documentation
- Replace references to `f()` with [f()].
- Document default vaules in signature, not in `@param` or elsewhere
- If `coords` has wrong dimensions `get_biomass()` should throw an error HOT 2
- Support for the component ratio method HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from allodb.