Skip to content

Commit

Permalink
retry mechanism for total and fix overwrite
Browse files Browse the repository at this point in the history
  • Loading branch information
axif0 committed Oct 25, 2024
1 parent 328d916 commit a627523
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 7 deletions.
20 changes: 19 additions & 1 deletion src/scribe_data/cli/total.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@
"""

from SPARQLWrapper import JSON
from urllib.error import HTTPError
from http.client import IncompleteRead

from scribe_data.utils import (
LANGUAGE_DATA_EXTRACTION_DIR,
Expand Down Expand Up @@ -244,7 +246,23 @@ def get_total_lexemes(language, data_type, doPrint=True):

sparql.setQuery(query)
sparql.setReturnFormat(JSON)
results = sparql.query().convert()
try_count = 0
max_retries = 2
results = None

while try_count <= max_retries and results is None:
try:
results = sparql.query().convert()
except HTTPError as http_err:
print(f"HTTPError occurred: {http_err}")
except IncompleteRead as read_err:
print(f"Incomplete read error occurred: {read_err}")
try_count += 1

if results is None and try_count <= max_retries:
print("The query will be retried..")
elif results is None:
print("Query failed after retries.")

# Check if the query returned any results.
if (
Expand Down
12 changes: 6 additions & 6 deletions src/scribe_data/wikidata/query_data.py
Original file line number Diff line number Diff line change
Expand Up @@ -181,13 +181,13 @@ def query_data(
for file in existing_files:
file.unlink()

# elif choice in ["k", "K"]:
# timestamp = datetime.now().strftime("%Y_%m_%d_%H_%M_%S")
# file_name = f"{target_type}_{timestamp}.json"
# elif choice in ["k", "K"]:
# timestamp = datetime.now().strftime("%Y_%m_%d_%H_%M_%S")
# file_name = f"{target_type}_{timestamp}.json"

else:
print(f"Skipping update for {lang} {target_type}.")
continue
else:
print(f"Skipping update for {lang} {target_type}.")
break

print(f"Querying and formatting {lang} {target_type}")

Expand Down

0 comments on commit a627523

Please sign in to comment.