Skip to content

Commit

Permalink
Wording and typo
Browse files Browse the repository at this point in the history
  • Loading branch information
kklemon committed Jun 6, 2024
1 parent ccff4f5 commit 69cddca
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/penai/svg.py
Original file line number Diff line number Diff line change
Expand Up @@ -770,7 +770,7 @@ def remove_shape(self, shape_id: str) -> None:
raise AssertionError(f"Shape with id {shape_id} was not removed correctly.")

def remove_elements_with_no_visible_content(self) -> None:
# Sort the shapes by depth in shapes, so that we start with the deepest shapes
# Sort the shapes by depth in shapes, so that we start with the deepest shapes.
# Otherwise we may delete a parent shape before its children, thus decouple the children from the tree
# which will lead to weird behavior (i.e. lxml will assign arbitrary namespace names) and errors.
# We could, of course, also detect these relationships and only remove invisible parents,
Expand Down
2 changes: 1 addition & 1 deletion test/penai/test_svg.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ def test_removing_shapes_without_content(
chrome_svg_renderer: BaseSVGRenderer,
log_dir: Path,
) -> None:
# TODO: this adds a relatively large delay to the tests.
# TODO: this adds a relatively large overhead to the tests.
# We should consider reducing the number of files or pages we test on.
for file in example_project.load().files.values():
for page in file.pages.values():
Expand Down

0 comments on commit 69cddca

Please sign in to comment.