Skip to content
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

Output of arxiv_search is now a data frame #7

Merged
merged 22 commits into from
Sep 7, 2014
Merged

Output of arxiv_search is now a data frame #7

merged 22 commits into from
Sep 7, 2014

Conversation

kbroman
Copy link
Member

@kbroman kbroman commented Sep 7, 2014

  • Added a bunch of stuff to clean up the results
  • Added a bunch of tests
  • Output of arxiv_search() and arxiv_count() also include an attribute "search_info" with search parameters and the time of the search.

The key element of the package is now complete, so I've made this version 0.2.

 - also, strip trailing semicolons if necessary; saw that in one
   instance with multiple DOI links
 - also added tests

 - helper function get_key: made version for plain list
   as well as the version for a list of lists, which is now called
   get_key_ll
 - also tests

 - also revised clean_record to have separator arg
 - separator seemed more clear, but sep is more standard in R
   (e.g., in paste)
 - include argument output_format that can be data.frame or list, but
   data.frame is the default

 - result includes an attribute "search_info" with search
   parameters (and the time at which the search was completed)
 - attribute to results of arxiv_search() and arxiv_count()
 - Was pasting affiliations from multiple authors; if all missing, we
   end up with a string with one fewer vertical bars than there are
   authors. Better to just make it an empty string.
kbroman added a commit that referenced this pull request Sep 7, 2014
Output of arxiv_search is now a data frame
@kbroman kbroman merged commit 8f154eb into ropensci:master Sep 7, 2014
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.

1 participant