Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | datacite: fix a raw name constraint violation | Martin Czygan | 2020-04-20 | 1 | -0/+8 | |
|/ | | | | | | | It was possible that contribs got added which had no raw name. One example would be a name consisting of whitespace only. This fix adds a final check for this case. | |||||
* | consistently use raw string prefix for regex | Bryan Newbold | 2020-04-17 | 1 | -1/+1 | |
| | ||||||
* | pubmed: use untranslated title if translated not available | Bryan Newbold | 2020-04-01 | 1 | -0/+6 | |
| | | | | | | | The primary motivation for this change is that fatcat *requires* a non-empty title for each release entity. Pubmed/Medline occasionally indexes just a VenacularTitle with no ArticleTitle for foreign publications, and currently those records don't end up in fatcat at all. | |||||
* | importers: replace newlines in get_text() strings | Bryan Newbold | 2020-04-01 | 4 | -23/+25 | |
| | ||||||
* | importers: more string/get_text swaps | Bryan Newbold | 2020-03-28 | 3 | -27/+27 | |
| | | | | See previous pubmed commit for details. | |||||
* | pubmed: bunch of .get_text() instead of .string | Bryan Newbold | 2020-03-28 | 1 | -12/+12 | |
| | | | | | | | | | | | Yikes! Apparently when a tag has child tags, .string will return None instead of all the strings. .get_text() returns all of it: https://www.crummy.com/software/BeautifulSoup/bs4/doc/#get-text https://www.crummy.com/software/BeautifulSoup/bs4/doc/#string I've things like identifiers as .string, when we expect only a single string inside. | |||||
* | Merge pull request #53 from EdwardBetts/spelling | bnewbold | 2020-03-27 | 3 | -7/+7 | |
|\ | | | | | Correct spelling mistakes | |||||
| * | Correct spelling mistakes | Edward Betts | 2020-03-27 | 3 | -7/+7 | |
| | | ||||||
* | | datacite: nameIdentifier corner case | Bryan Newbold | 2020-03-26 | 1 | -1/+2 | |
| | | | | | | | | | | | | | | | | | | Works around a bug in production: AttributeError: 'NoneType' object has no attribute 'replace' (datacite.py:724) NOTE: there are no tests for this code path | |||||
* | | jalc: avoid meaningless pages values | Bryan Newbold | 2020-03-23 | 1 | -4/+8 | |
| | | ||||||
* | | datacite: add year sanity restrictions | bnewbold | 2020-03-23 | 1 | -0/+7 | |
| | | | | | | | | | | | | | | | | | | Example of entities with bogus years: https://fatcat.wiki/release/search?q=doi_registrar%3Adatacite+year%3A%3E2100 We can do a clean-up task, but first need to prevent creation of new bad metadata. | |||||
* | | pubmed: handle multiple ReferenceList | Bryan Newbold | 2020-03-20 | 1 | -1/+4 | |
| | | | | | | | | | | | | | | This resolves a situation noticed in prod where we were only importing/updating a single reference per article. Includes a regression test. | |||||
* | | pubmed: update many more metadata fields | Bryan Newbold | 2020-03-19 | 1 | -0/+22 | |
| | | | | | | | | | | | | | | In particular, with daily updates in most cases the DOI will be registered first, then the entity updated with PMID when that is available. Often the pubmed metadata will be more complete, with abstracts etc, and we'll want those improvements. | |||||
* | | crossref: skip stub OUP title | Bryan Newbold | 2020-03-19 | 1 | -0/+8 | |
| | | | | | | | | | | | | It seems like OUP pre-registers DOIs with this place-holder title, then updates the Crossref metdata when the paper is actually published. We should wait until the real title is available before creating an entity. | |||||
* | | Merge branch 'martin-kafka-bs4-import' into 'master' | Martin Czygan | 2020-03-10 | 2 | -1/+66 | |
|\ \ | |/ |/| | | | | | pubmed and arxiv harvest preparations See merge request webgroup/fatcat!28 | |||||
| * | common: use smaller batch size since XML parsing may be slow | Martin Czygan | 2020-03-10 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | Address kafka tradeoff between long and short time-outs. Shorter time-outs would facilitate > consumer group re-balances and other consumer group state changes [...] in a reasonable human time-frame. | |||||
| * | pubmed ftp harvest and KafkaBs4XmlPusher | Martin Czygan | 2020-02-19 | 2 | -1/+66 | |
| | | | | | | | | | | | | | | * add PubmedFTPWorker * utils are currently stored alongside pubmed (e.g. ftpretr, xmlstream) but may live elsewhere, as they are more generic * add KafkaBs4XmlPusher | |||||
* | | add some more domain/rel URL mappings | Bryan Newbold | 2020-02-22 | 1 | -0/+9 | |
| | | ||||||
* | | Merge branch 'bnewbold-shadow-import' | Bryan Newbold | 2020-02-19 | 2 | -0/+196 | |
|\ \ | ||||||
| * | | remove arabesque short wayback URL hack | Bryan Newbold | 2020-02-14 | 1 | -6/+0 | |
| | | | ||||||
| * | | improve shadow import file url cleanup path | Bryan Newbold | 2020-02-13 | 1 | -2/+12 | |
| | | | | | | | | | | | | Should probably be refactored out in to shared cleanup code. | |||||
| * | | shadow import fixes from QA testing | Bryan Newbold | 2020-02-13 | 1 | -0/+6 | |
| | | | ||||||
| * | | shadow import: more filtering of file_meta fields | Bryan Newbold | 2020-02-13 | 1 | -0/+10 | |
| | | | ||||||
| * | | basic shadow importer | Bryan Newbold | 2020-02-13 | 2 | -0/+176 | |
| |/ | ||||||
* | | ingest import: fix edit_extra path | Bryan Newbold | 2020-02-18 | 1 | -1/+1 | |
| | | ||||||
* | | ingest importer: edit_extra is a top-level key | Bryan Newbold | 2020-02-18 | 1 | -1/+1 | |
| | | ||||||
* | | ingest import: allow short version of corpus names | Bryan Newbold | 2020-02-18 | 1 | -0/+3 | |
| | | ||||||
* | | ingest importer: pass through link rel | Bryan Newbold | 2020-02-18 | 1 | -1/+6 | |
|/ | ||||||
* | check ingest_request_source existance for SPN as well as ingest | Bryan Newbold | 2020-02-06 | 1 | -0/+3 | |
| | ||||||
* | additional trusted link sources | Bryan Newbold | 2020-02-06 | 1 | -0/+3 | |
| | ||||||
* | add mag and s2 as trusted link sources | Bryan Newbold | 2020-02-06 | 1 | -1/+1 | |
| | ||||||
* | ingest worker: handle missing ingest_request_source | Bryan Newbold | 2020-02-06 | 1 | -0/+3 | |
| | | | | | Seeing a bunch of these due to re-ingests not including this field because of an earlier persist bug. | |||||
* | datacite: prevent none | Martin Czygan | 2020-01-31 | 1 | -1/+1 | |
| | ||||||
* | datacite: name shall not be None | Martin Czygan | 2020-01-31 | 1 | -1/+1 | |
| | ||||||
* | datacite: add exception for https://www.micropublication.org/ | Martin Czygan | 2020-01-31 | 1 | -0/+5 | |
| | ||||||
* | datacite: do not skip records w/o date | Martin Czygan | 2020-01-31 | 1 | -2/+1 | |
| | ||||||
* | datacite: improve docstring | Martin Czygan | 2020-01-31 | 1 | -4/+4 | |
| | ||||||
* | datacite: improve date handling and minor tweak | Martin Czygan | 2020-01-30 | 1 | -19/+42 | |
| | | | | | | | | | | | | | Records from https://www.micropublication.org/ did not have a date in FC, although raw data contained date strings - they were not using the finer-grained "attributes.date" but "attributes.published" and/or "attributes.publicationYear". Support for those fields has been added, including a test case. During this test (#30) a processing gap for names became clear (author may have "given_name" and "surname", but no "name"). This bug has been fixed, too. | |||||
* | fix KafkaError worker reporting for partition errors | Bryan Newbold | 2020-01-29 | 1 | -1/+1 | |
| | ||||||
* | fix trivial typo in file importer | Bryan Newbold | 2020-01-20 | 1 | -1/+1 | |
| | ||||||
* | ingest: improve tests, support old ingest results | Bryan Newbold | 2020-01-15 | 1 | -3/+12 | |
| | ||||||
* | update ingest worker for schema tweaks | Bryan Newbold | 2020-01-15 | 1 | -8/+15 | |
| | | | | | | Should be backwards compatible with old ingest results. Fixed a bug with glutton ident detection. | |||||
* | ingest: allow more sources to auto-import | Bryan Newbold | 2020-01-15 | 1 | -1/+2 | |
| | ||||||
* | datacite: skip records without a doi | Martin Czygan | 2020-01-13 | 1 | -0/+4 | |
| | ||||||
* | datacite: add entry to license slug map | Martin Czygan | 2020-01-09 | 1 | -0/+1 | |
| | ||||||
* | datacite: ignore known unknown values in resourceType* | Martin Czygan | 2020-01-09 | 1 | -2/+2 | |
| | ||||||
* | datacite: abstracts may be strings or list of strings | Martin Czygan | 2020-01-09 | 1 | -2/+15 | |
| | ||||||
* | datacite: improve license_slug handling | Martin Czygan | 2020-01-09 | 1 | -60/+101 | |
| | ||||||
* | datacite: add 'Unknown' to blacklist | Martin Czygan | 2020-01-09 | 1 | -1/+5 | |
| | ||||||
* | datacite: get rid of schemaVersion | Martin Czygan | 2020-01-09 | 1 | -3/+0 | |
| |