Skip to content

bug: Add catch for military time in CLI #265

@JMCulhane

Description

@JMCulhane

Bug Report

CIB Mango Tree version:

v0.9.0

Current behavior:

Currently, the CLI does not format military time values as of type "Time." It is instead listed as type: 'Identifier.'

Expected behavior:

The CLI should format military time types to be of type "Time" when parsing data sets.

Steps to reproduce:

Using this data sheet: https://drive.google.com/file/d/17xfpGmnJsCWZeFyuNKR9zQPZm--zIa0D/view?usp=drive_link, import the set. Be sure to modify the import with comma separators in the CLI, as the dataset is a bit finnicky.

Use the debugger, or include print statements for the "infer_series_semantic" function via the following path: mango-tango-cli/preprocessing/series_semantic.py to see how the data is formatted.

Related code:

Here is the full function being executed for the semantic inference.

def infer_series_semantic(
series: pl.Series, *, threshold: float = 0.8, sample_size=100
):
for semantic in all_semantics:
if semantic.check(series, threshold=threshold, sample_size=sample_size):
return semantic
return None

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions