Amanda Cameron 1fb464235a
chore: Table chunking (#1540)
This change is adding to our `add_chunking_strategy` logic so that we
are able to chunk Table elements' `text` and `text_as_html` params. In
order to keep the functionality under the same `by_title` chunking
strategy we have renamed the `combine_under_n_chars` to
`max_characters`. It functions the same way for the combining elements
under Title's, as well as specifying a chunk size (in chars) for
TableChunk elements.

*renaming the variable to `max_characters` will also reflect the 'hard
max' we will implement for large elements in followup PRs


Additionally -> some lint changes snuck in when I ran `make tidy` hence
the minor changes in unrelated files :)

TODO:
 add unit tests
--> note: added where I could to unit tests! Some unit tests I just
clarified that the chunking strategy was now 'by_title' because we don't
have a file example that has Table elements to test the
'by_num_characters' chunking strategy
  update changelog

To manually test:
```
In [1]: filename="example-docs/example-10k.html"

In [2]: from unstructured.chunking.title import chunk_table_element

In [3]: from unstructured.partition.auto import partition

In [4]: elements = partition(filename)

# element at -2 happens to be a Table, and we'll get chunks of char size 4 here
In [5]: chunks = chunk_table_element(elements[-2], 4)

# examine text and text_as_html params
ln [6]: for c in chunks:
                    print(c.text)
                    print(c.metadata.text_as_html)
```

---------

Co-authored-by: Yao You <theyaoyou@gmail.com>
2023-10-03 09:40:34 -07:00

193 lines
5.9 KiB
Python

import os
import pathlib
from unstructured.chunking.title import chunk_by_title
from unstructured.documents.elements import Table, TableChunk, Title
from unstructured.partition.json import partition_json
from unstructured.partition.odt import partition_odt
from unstructured.staging.base import elements_to_json
DIRECTORY = pathlib.Path(__file__).parent.resolve()
EXAMPLE_DOCS_DIRECTORY = os.path.join(DIRECTORY, "..", "..", "..", "example-docs")
def test_partition_odt_from_filename():
filename = os.path.join(EXAMPLE_DOCS_DIRECTORY, "fake.odt")
elements = partition_odt(filename=filename)
assert elements == [
Title("Lorem ipsum dolor sit amet."),
Table(
text="\nHeader row Mon Wed"
" Fri\nColor Blue"
" Red Green\nTime 1pm"
" 2pm 3pm\nLeader "
"Sarah Mark Ryan",
),
]
for element in elements:
assert element.metadata.filename == "fake.odt"
def test_partition_odt_from_filename_with_metadata_filename():
filename = os.path.join(EXAMPLE_DOCS_DIRECTORY, "fake.odt")
elements = partition_odt(filename=filename, metadata_filename="test")
assert all(element.metadata.filename == "test" for element in elements)
def test_partition_odt_from_file():
filename = os.path.join(EXAMPLE_DOCS_DIRECTORY, "fake.odt")
with open(filename, "rb") as f:
elements = partition_odt(file=f)
assert elements == [
Title("Lorem ipsum dolor sit amet."),
Table(
text="\nHeader row Mon Wed"
" Fri\nColor Blue"
" Red Green\nTime 1pm"
" 2pm 3pm\nLeader "
"Sarah Mark Ryan",
),
]
def test_partition_odt_from_file_with_metadata_filename():
filename = os.path.join(EXAMPLE_DOCS_DIRECTORY, "fake.odt")
with open(filename, "rb") as f:
elements = partition_odt(file=f, metadata_filename="test")
assert all(element.metadata.filename == "test" for element in elements)
def test_partition_odt_from_filename_exclude_metadata():
filename = os.path.join(EXAMPLE_DOCS_DIRECTORY, "fake.odt")
elements = partition_odt(filename=filename, include_metadata=False)
for i in range(len(elements)):
assert elements[i].metadata.to_dict() == {}
def test_partition_odt_from_file_exclude_metadata():
filename = os.path.join(EXAMPLE_DOCS_DIRECTORY, "fake.odt")
with open(filename, "rb") as f:
elements = partition_odt(file=f, include_metadata=False)
for i in range(len(elements)):
assert elements[i].metadata.to_dict() == {}
def test_partition_odt_metadata_date(
mocker,
filename="example-docs/fake.odt",
):
mocked_last_modification_date = "2029-07-05T09:24:28"
mocker.patch(
"unstructured.partition.odt.get_last_modified_date",
return_value=mocked_last_modification_date,
)
elements = partition_odt(
filename=filename,
)
assert elements[0].metadata.last_modified == mocked_last_modification_date
def test_partition_odt_with_custom_metadata_date(
mocker,
filename="example-docs/fake.odt",
):
mocked_last_modification_date = "2029-07-05T09:24:28"
expected_last_modification_date = "2020-07-05T09:24:28"
mocker.patch(
"unstructured.partition.odt.get_last_modified_date",
return_value=mocked_last_modification_date,
)
elements = partition_odt(
filename=filename,
metadata_last_modified=expected_last_modification_date,
)
assert elements[0].metadata.last_modified == expected_last_modification_date
def test_partition_odt_from_file_metadata_date(
mocker,
filename="example-docs/fake.odt",
):
mocked_last_modification_date = "2029-07-05T09:24:28"
mocker.patch(
"unstructured.partition.odt.get_last_modified_date_from_file",
return_value=mocked_last_modification_date,
)
with open(filename, "rb") as f:
elements = partition_odt(
file=f,
)
assert elements[0].metadata.last_modified == mocked_last_modification_date
def test_partition_odt_from_file_with_custom_metadata_date(
mocker,
filename="example-docs/fake.odt",
):
mocked_last_modification_date = "2029-07-05T09:24:28"
expected_last_modification_date = "2020-07-05T09:24:28"
mocker.patch(
"unstructured.partition.odt.get_last_modified_date_from_file",
return_value=mocked_last_modification_date,
)
with open(filename, "rb") as f:
elements = partition_odt(file=f, metadata_last_modified=expected_last_modification_date)
assert elements[0].metadata.last_modified == expected_last_modification_date
def test_partition_odt_with_json(
filename=os.path.join(EXAMPLE_DOCS_DIRECTORY, "fake.odt"),
):
elements = partition_odt(filename=filename, include_metadata=True)
test_elements = partition_json(text=elements_to_json(elements))
assert len(elements) == len(test_elements)
for i in range(len(elements)):
assert elements[i] == test_elements[i]
def test_add_chunking_strategy_on_partition_odt(
filename="example-docs/fake.odt",
):
elements = partition_odt(filename=filename)
chunk_elements = partition_odt(filename, chunking_strategy="by_title")
chunks = chunk_by_title(elements)
assert chunk_elements != elements
assert chunk_elements == chunks
def test_add_chunking_strategy_on_partition_odt_non_default():
filename = os.path.join(EXAMPLE_DOCS_DIRECTORY, "fake.odt")
elements = partition_odt(filename=filename)
chunk_elements = partition_odt(
filename,
chunking_strategy="by_title",
max_characters=7,
combine_text_under_n_chars=5,
)
chunks = chunk_by_title(
elements,
max_characters=7,
combine_text_under_n_chars=5,
)
for chunk in chunk_elements:
if isinstance(chunk, TableChunk):
assert len(chunk.text) <= 7
assert chunk_elements != elements
assert chunk_elements == chunks