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

Create update_dblp_auto.py #7

Merged
merged 3 commits into from
Oct 24, 2023
Merged

Create update_dblp_auto.py #7

merged 3 commits into from
Oct 24, 2023

Conversation

lancercat
Copy link
Contributor

Make things lazier by adding a semiauto updater:
It only needs manul intervention when there is more than one choices that are not from arxiv,

Make things lazier.
@lancercat
Copy link
Contributor Author

4sec is still too short and still triggers 429 I will find the value by trial and error.

@volkm
Copy link
Owner

volkm commented Oct 23, 2023

Thanks, I very much like this idea!
It seems to me that the code is very similar to update_from_dblp.py. Would it possible to integrate your changes into the existing file and maybe add a cmdline flag --disable-auto to disable the automatic selection? I imagine that for most users, this automatic selection is preferred and we should make it the default.

Adds an option to select auto/manual mode.
@lancercat
Copy link
Contributor Author

Done adding an option, not tested the manual mode yet as I am crawling to fix my 300+ citation entries...
Well, my suggestion is we keep both, and we merge them when the new code looks robust enough.

@volkm
Copy link
Owner

volkm commented Oct 24, 2023

I integrated your suggestions into update_from_dblp and enabled the automatic selection by default. It can be disabled with --disable-auto. Additional cmdline flags are --include-arxiv to explicitly keep arXiv entries in the search results and --sleep-time to set the waiting time between requests (in order to prevent too many requests).

@volkm volkm merged commit 8f6cce8 into volkm:master Oct 24, 2023
4 checks passed
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.

2 participants