-
Notifications
You must be signed in to change notification settings - Fork 8
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
oami_pmc_pmcid_import should loop over the PMCIDs found by oa-pmc-ids #111
Comments
Just ran
which gave
This error is the subject of #84 - the point here in #111 is that such an individual error should not stop the processing of other articles, but it currently does. Moreover, the current handling of the looping does not allow me to find out which article had the "Unknown, possibly non-free license: " and whether that statement is correct. |
I assume we can use xargs. |
Does using a read loop fix the problem? If so, i'll add it to she shell script. ./oa-pmc-ids --from 2013-10-14 --until 2013-10-15 | tr ' ' '\n' | while read -r; do echo $REPLY | ./oami_pmc_pmcid_import; done; |
Running it now. Looks good so far. |
No problem so far.
now. |
Got stuck with
so we will have to use timeout perhaps. |
Just set up a cron job
with
|
Daniel, if i put the above code into the repository, can the bug be closed? |
I am now using a cronjob
where oami_pmc_pmcid_import.sh is
This works fine, so there is no need for your workaround in #111 (comment) , and I am closing this issue. |
For proper logging: 12 6 * * * cd ~/open-access-media-importer; sh oami_pmc_pmcid_import.sh 2>&1 | tee -a oami_pmc_pmcid_import.tee |
Moved here from #94 (comment) .
The text was updated successfully, but these errors were encountered: