Repository: acl-org/acl-anthology

Issue:
Please re-open #978 (Erroneous link in online documentation)
Reason:

The sentence in question now reads `Our scripts use the [lxml.de](lxml.de) library to parse the XML.`, which results in the invalid local link https://www.aclweb.org/anthology/info/development/lxml.de instead of https://lxml.de.

Some explanations:

@akoehn Your perception of my original message made me think (surprisingly) long and hard. I get that stating one's own intentions does not retroactively heal a rude or confrontational impression evoked on the side of a reader or listener.
I had thought of a terse style as wasting the least amount of attention for everyone involved, as a form of politeness, actually - well, you live and learn.

@mjpost Of course this is innocuous and hardly worth mentioning. Most probably, the page was drafted in 'airplane mode' and the placeholder-URI simply stayed in there.

Disclaimer:

No, I do not intend to wisecrack on every typo I might find in the future. What led me here was reading a string of articles ([Quanta Magazine](https://www.quantamagazine.org/)) on AI and NLP seeking inspiration (i.e., shoplifting ideas) for algorithmic composition (yes, the musical kind). This should explain why the probability of me contributing anything substantial to this repository is arbitrarily close to zero and cloning the whole shebang to submit a proper pull request would be slightly over the top.



Discussion:

Resolving patch:
diff --git a/bin/add_revision.py b/bin/add_revision.py
--- a/bin/add_revision.py
+++ b/bin/add_revision.py
@@ -114,7 +114,6 @@ def maybe_copy(file_from, file_to):
     validate_file_type(input_file_path)
 
     collection_id, volume_id, paper_id = deconstruct_anthology_id(args.anthology_id)
-    venue_name = collection_id.split(".")[1]
     paper_extension = args.path.split(".")[-1]
 
     # The new version
@@ -126,6 +125,7 @@ def maybe_copy(file_from, file_to):
     # Files for old-style IDs are stored under anthology-files/pdf/P/P19/*
     # Files for new-style IDs are stored under anthology-files/pdf/2020.acl/*
     if is_newstyle_id(args.anthology_id):
+        venue_name = collection_id.split(".")[1]
         output_dir = os.path.join(args.anthology_dir, "pdf", venue_name)
     else:
         output_dir = os.path.join(
diff --git a/bin/create_hugo_pages.py b/bin/create_hugo_pages.py
--- a/bin/create_hugo_pages.py
+++ b/bin/create_hugo_pages.py
@@ -113,7 +113,9 @@ def create_volumes(srcdir, clean=False):
                 {
                     "anthology_id": anthology_id,
                     "title": entry["title"],
-                    "aliases": [paper_dir,],
+                    "aliases": [
+                        paper_dir,
+                    ],
                 },
                 default_flow_style=False,
                 stream=f,
diff --git a/bin/tacl_cl_parser.py b/bin/tacl_cl_parser.py
--- a/bin/tacl_cl_parser.py
+++ b/bin/tacl_cl_parser.py
@@ -400,9 +400,7 @@ def sort_papers_by_page(paper_tuple):
         return startpage
 
     paper_id = 1  # Stupid non-enumerate counter because of "Erratum: " papers interleaved with real ones.
-    for papernode, pdf_path, issue_info, issue in sorted(
-        papers, key=sort_papers_by_page
-    ):
+    for papernode, pdf_path, issue_info, issue in sorted(papers, key=sort_papers_by_page):
         issue = issue or "1"
         if issue_info != previous_issue_info:
             # Emit the new volume info before the paper.


Test patch: