Skip to content

Commit

Permalink
Make Entity.layers a little more magical by checking if spans and box…
Browse files Browse the repository at this point in the history
…es exist
  • Loading branch information
josephcc committed Mar 14, 2024
1 parent 6a0a4a2 commit 460e165
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion papermage/magelib/entity.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,13 @@ def id(self, id: int) -> None:
def __getattr__(self, name: str) -> List["Entity"]:
"""This Overloading is convenient syntax since the `entity.layer` operation is intuitive for folks."""
try:
return self.intersect_by_span(name=name)
if len(self.spans) > 0:
intersection = self.intersect_by_span(name=name)
if len(intersection) == 0 and len(self.boxes) > 0:
intersection = self.intersect_by_box(name=name)
return intersection
else:
return self.intersect_by_box(name=name)
except ValueError:
# maybe users just want some attribute of the Entity object
return self.__getattribute__(name)
Expand Down

0 comments on commit 460e165

Please sign in to comment.