ropensci / openalexR

Getting bibliographic records from OpenAlex
https://docs.ropensci.org/openalexR/
Other
91 stars 20 forks source link

Release openalexR 1.2.0 #138

Closed massimoaria closed 1 year ago

massimoaria commented 1 year ago

Prepare for release:

Submit to CRAN:

Wait for CRAN...

jeroen commented 1 year ago

Can you also try to fix the error in A_Brief_Introduction_to_openalexR.Rmd: https://github.com/r-universe/ropensci/actions/runs/5679193215/job/15390913797. Thank you! 🙏

trangdata commented 1 year ago

Thank you for pointing this out @jeroen. 🌻 I took a look at the vignette and it seems that this may have to do with the recent update in OpenAlex's new authors rollout. I sent a ticket to the OpenAlex team.

Specifically, these are not working:

https://api.openalex.org/authors?filter=last_known_institution.id%3Ahttps%3A%2F%2Fopenalex.org%2FI4200000001 https://api.openalex.org/authors?filter=last_known_institution.country_code%3ACA (only return 7 authors)

trangdata commented 1 year ago

Looks like in the transition to the new author system, last_known_institutions for authors is lost. They're all currently null for the new authors. The OpenAlex team is working on fixing this.

massimoaria commented 1 year ago

@trangdata Unfortunately, tests now return the following errors:

==> devtools::test()

ℹ Testing openalexR Thank you for using openalexR! To acknowledge our work, please cite the package by calling citation("openalexR"). ✔ | F W S OK | Context ⠦ | 7 | oa_fetch Requesting url: https://api.openalex.org/works?filter=doi%3A10.1371%2Fjournal.pone.0266781%7C10.1371%2Fjournal.pone.0267149&select=doi%2Cid%2Ccited_by_count%2Ctype Getting 1 page of results with a total of 2 records... ✖ | 2 50 | oa_fetch [29.1s]
────────────────────────────────────────────────────────────────── Failure (test-oa_fetch.R:212:3): oa_fetch can combine (OR) more than 50 DOIs in a filter nrow(many_doi_results) (actual) not equal to length(valid_dois) (expected).

actual: 50 expected: 51

Failure (test-oa_fetch.R:276:3): oa_fetch can combine (OR) more than 50 ORCIDs in a filter nrow(many_orcid_results) >= length(valid_orcids) is not TRUE

actual: FALSE expected: TRUE ────────────────────────────────────────────────────────────────── ✔ | 3 | oa_ngrams [0.8s]
⠏ | 0 | oa_query Identifier is missing, please specify filter or search argument. ✔ | 3 | oa_query ✔ | 11 | oa_snowball [12.8s]
✔ | 2 | oa2bibliometrix [1.6s]
✔ | 10 | oa2df [1.4s]
✔ | 6 | simplify [2.3s]
✔ | 15 | utils

══ Results ═══════════════════════════════════════════════════════ Duration: 48.1 s

[ FAIL 2 | WARN 0 | SKIP 0 | PASS 100 ]