Comments (23)
Bulk analysis looks basic, but I think is fine.
I'll add some text to give more details.
FWIW. my vision is:
- "get_started" for a short overview;
- "elephant" to give examples of what you can do with the data for one taxon;
- "bulk analaysis" show how to use the package to get data for many taxa;
from rcites.
In Get started with rcites and Speciesplus information about the African elephant I quickly drafted what it could look like. tell how you like it.
from rcites.
Thanks! Looks good!
I'm revisiting several aspects of the package to better use the API, I'll let you what are the new features that should be included in the vignette!
from rcites.
Vignettes look great. I'll wait to have the full package with pagination to test the package and update them.
from rcites.
In all vignette when we use the false token, i.e. token = "8QW6Qgh57sBG2k0gtt"
, the code chunk must be set to eval=FALSE
, for instance:
```{r, eval = FALSE}
res1 <- spp_taxonconcept(query_taxon = "Loxodonta africana", token = "8QW6Qgh57sBG2k0gtt")
```
or
```{r, eval = FALSE}
set_token("8QW6Qgh57sBG2k0gtt")
```
Otherwise,when we build the vignette, we get the following error:
Error: processing vignette 'rcites-vignette.Rmd' failed with diagnostics:
Unauthorized (HTTP 401).
from rcites.
@ibartomeus yes sure, we should wait for all functions and the 2nd review (=> potential change) to be done before doing the documentation.
@KevCaz thanks for the info, done.
from rcites.
So we now have 3 vignettes:
- Get started
- Bulk analysis
- Elephant
Get started is pretty much done. Bulk analysis looks basic, but I think is fine. Elephant needs some more work, as this should explain a whole workflow with the package and make the outputs understandable.
I will continue working on it this week. @ibartomeus do you have some time, too?
from rcites.
Yes, sorry for my lack of time. I'll run everything to see how it works and any potential improvement and work on the vignettes a bit.
from rcites.
Alright, I just had a look at all of the three and cleaned up "elephant". I dont think you need to give "bulk analysis" more text, as it pretty clearly shows how to do/code this. The only thing that could be added, as the same with "elephant" now, is some explanation about the outputs.
Regarding the development of "elephant" as case study: Your idea with the map of trade directions is great and should be doable. However, in light of the time period gone past since the reviews came in, I think we should be good to go and add more details and use examples bit by bit. Like I think the package functioning is advanced quite a lot so we should not wait too long just because of some use examples ...
from rcites.
I do agree that the time has come to send the revised package back to the reviewers. I'll have a last check latter today. Let say we submit our revised Monday (or before if you are available)?
from rcites.
Perfect.
As far as I see you can do your last check (@ibartomeus you need some more time if you want to add something before re-submission? however can do more work and add use examples at any time ;-) ) and post the answer to reviewers thereafter. If you prefer, also I can do this.
Then we need to wait for their response and hopefully approval. Thereafter, we will need to do some more things in order to have the package with rOpenSci ...
from rcites.
It seems like @ibartomeus doenst have too much time these days (which is fine). So @KevCaz did you do the last check? Any issues that came up and you need help with?
If not, would you rebuild the website so we can post the answer text to the reviewers?
from rcites.
I can do it today (and I'll be happy to look through everything, even if my comments will be very minor or non-existing, giving your amazing job)! Sorry again.
from rcites.
No worries at al @ibartomeus! We all have other duties to deal with, so it is fine. Special thanks in this regard to @KevCaz for dealing with all the technical issues!
So I guess we are good to go and get the package resubmitted asap.
from rcites.
What the 'load_all()' function is doing in line 53 of the get_started vignette? it fails for me ('Error in load_all() : could not find function "load_all"').
Ok, it knits correctly, but if fails when ran as stand alone.
from rcites.
@ibartomeus I used it for some reasons (bad reasons) it should be replaced by library(rcites)
.
from rcites.
Ok, I'll do that!
from rcites.
Can we also remove !-- https://cran.r-project.org/web/packages/httr/vignettes/secrets.html --
from line 24 on elephant.rmd?
from rcites.
from rcites.
I found one bug in Bulk analysis:
res_cms <- spp_taxonconcept("", taxonomy = "CMS")
Retrieving info from page 6 / 6
Done!
Error in do.call(rbind, y[[name]]) : second argument must be a list
dim(res_cms$general)
NULL
from rcites.
Ok, other than the bug above, I corrected a few typos and made a few clarifications. All pushed. @KevCaz make a quick look to check I didn't add anything wrong and let's submit it! The package looks impressive!
from rcites.
I'll have the final look, I was suppose to do that during the week-end but did not find the time to do so!
Thanks @ibartomeus!
from rcites.
All fixed, website rebuilt. I think we can close this for now and open a new issue when the time of reviewing the vignettes will come.
from rcites.
Related Issues (20)
- Infinite recursion error when using non-interactively without token HOT 4
- Installation on covr failed in Travis HOT 1
- spp_cites_legislation() fails to return clean output for some taxa HOT 5
- Not Found (HTTP 404): spp_cites_legislation bulk analysis HOT 5
- pause feature for spp_*() functions HOT 1
- Website not deployed HOT 1
- Error in getting distribution data for species endemic to a single region HOT 3
- Change how classes are tested HOT 1
- Use GH actions instead of Travis HOT 1
- speed up tests with vcr
- test print methods properly HOT 2
- Precompute vignettes
- spp_*_legilation() sometime returns (apparently) unrelated info HOT 1
- console outputs and errors HOT 1
- Current issue on some GHA env for http request
- Report request status via `warn_for_status()` rather than `stop_for_status()` HOT 2
- Encode URLs HOT 3
- GHA setups keep failing on different environment HOT 3
- Synonyms info are not properly displayed
- pkgdown building failure HOT 6
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 rcites.