mirror of
https://github.com/ilri/csv-metadata-quality.git
synced 2024-11-25 07:10:17 +01:00
fix.py: Massive improvements
Use Python's str.strip() instead of kludgy regular expressions and use split/join to handle multi-value fields more cleanly.
This commit is contained in:
parent
801870e0ba
commit
ef5b8f7244
63
fix.py
63
fix.py
@ -2,7 +2,7 @@
|
|||||||
|
|
||||||
import pandas as pd
|
import pandas as pd
|
||||||
|
|
||||||
def fix_whitespace(value):
|
def fix_whitespace(field):
|
||||||
"""Fix whitespace issues.
|
"""Fix whitespace issues.
|
||||||
|
|
||||||
Return string with leading, trailing, and consecutive whitespace trimmed.
|
Return string with leading, trailing, and consecutive whitespace trimmed.
|
||||||
@ -10,62 +10,43 @@ def fix_whitespace(value):
|
|||||||
|
|
||||||
import re
|
import re
|
||||||
|
|
||||||
# Skip cells with missing values
|
# Skip fields with missing values
|
||||||
if pd.isna(value):
|
if pd.isna(field):
|
||||||
return
|
return
|
||||||
|
|
||||||
# Try to split multi-value cells on "||" separator
|
# Initialize an empty list to hold the cleaned values
|
||||||
#for value in cell.split('||'):
|
values = list()
|
||||||
|
|
||||||
# Check for leading whitespace
|
# Try to split multi-value field on "||" separator
|
||||||
pattern = re.compile(r'^\s+')
|
for value in field.split('||'):
|
||||||
|
# Strip leading and trailing whitespace
|
||||||
|
value = value.strip()
|
||||||
|
|
||||||
|
# Replace excessive whitespace (>2) with one space
|
||||||
|
pattern = re.compile(r'\s{2,}')
|
||||||
match = re.findall(pattern, value)
|
match = re.findall(pattern, value)
|
||||||
|
|
||||||
if len(match) > 0:
|
if len(match) > 0:
|
||||||
print('DEBUG: Leading whitespace')
|
print('DEBUG: Excessive whitespace')
|
||||||
value = re.sub(pattern, ' ', value)
|
value = re.sub(pattern, ' ', value)
|
||||||
|
|
||||||
# Check for leading whitespace in multi-value cells
|
# Save cleaned value
|
||||||
# SOME VALUE|| ANOTHER VALUE
|
values.append(value)
|
||||||
pattern = re.compile(r'\|\|\s+')
|
|
||||||
match = re.findall(pattern, value)
|
|
||||||
|
|
||||||
if len(match) > 0:
|
# Create a new field consisting of all values joined with "||"
|
||||||
print('DEBUG: Leading whitespace in multi-value cell')
|
new_field = '||'.join(values)
|
||||||
value = re.sub(pattern, '||', value)
|
|
||||||
|
|
||||||
# Check for trailing whitespace
|
return new_field
|
||||||
pattern = re.compile(r'\s+$')
|
|
||||||
match = re.findall(pattern, value)
|
|
||||||
|
|
||||||
if len(match) > 0:
|
|
||||||
print('DEBUG: Trailing whitespace')
|
|
||||||
value = re.sub(pattern, '', value)
|
|
||||||
|
|
||||||
# Check for trailing whitespace in multi-value cells
|
|
||||||
# SOME VALUE ||ANOTHER VALUE
|
|
||||||
pattern = re.compile(r'\s+\|\|')
|
|
||||||
match = re.findall(pattern, value)
|
|
||||||
|
|
||||||
if len(match) > 0:
|
|
||||||
print('DEBUG: Trailing whitespace in multi-value cell')
|
|
||||||
value = re.sub(pattern, '||', value)
|
|
||||||
|
|
||||||
return value
|
|
||||||
|
|
||||||
|
|
||||||
# Read all fields as strings so dates don't get converted from 1998 to 1998.0
|
# Read all fields as strings so dates don't get converted from 1998 to 1998.0
|
||||||
df = pd.read_csv('/home/aorth/Downloads/2019-07-26-Bioversity-Migration.csv', dtype=str)
|
#df = pd.read_csv('/home/aorth/Downloads/2019-07-26-Bioversity-Migration.csv', dtype=str)
|
||||||
#df = pd.read_csv('/tmp/quality.csv')
|
#df = pd.read_csv('/tmp/quality.csv', dtype=str)
|
||||||
#df = pd.read_csv('/tmp/omg.csv')
|
df = pd.read_csv('/tmp/omg.csv', dtype=str)
|
||||||
|
|
||||||
# Fix whitespace in all columns
|
# Fix whitespace in all columns
|
||||||
for column in df.columns.values.tolist():
|
for column in df.columns.values.tolist():
|
||||||
print(column)
|
print(f'DEBUG: {column}')
|
||||||
|
|
||||||
# Skip the id column
|
|
||||||
#if column == 'id':
|
|
||||||
# continue
|
|
||||||
|
|
||||||
df[column] = df[column].apply(fix_whitespace)
|
df[column] = df[column].apply(fix_whitespace)
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user