-
Notifications
You must be signed in to change notification settings - Fork 292
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
Look at DOIs for frontmatter #2724
Comments
I thought we do not ingest frontmatter anymore? |
We ingest it if it's there. Most of ACL had frontmatter. We need to check why we didn't assign DOIs. I am think we did in the past. |
You changed the logic floor them in the doi generation script recently -- maybe that has something to do with it? Did we change how frontmatters are represented in the xml and then had the doi generation script in am outdated state until you changed it? |
DOI ingestion is two steps:
What I changed is (2) which was broken because it assumed there was always a |
Actually, though, this reminds me that I also change the ingestion script (post-EMNLP) to always generate the |
Whatever the reasoning for always generating the |
We don't generate DOIs for the complete volume or frontmatter, and haven't for some time. If we want to, we just need to figure it out. I haven't had time to do this. See also #726. |
Re-upping this for this quarter—we should generate DOIs for front matter. This involves:
|
We didn't get DOIs in frontmatter for EMNLP 22 or ACL 23.
The text was updated successfully, but these errors were encountered: