Skip to content

Commit 8b5df06

Browse files
committed
Add warning to Renderer when trying to render without calling begin_rendering first
1 parent 9785c5e commit 8b5df06

File tree

3 files changed

+13
-3
lines changed

3 files changed

+13
-3
lines changed

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ description = "A high performance Python interface for communicating with RLBot
88
dynamic = ["version"]
99
requires-python = ">= 3.11"
1010
dependencies = [
11-
"rlbot_flatbuffers~=0.11.0",
11+
"rlbot_flatbuffers~=0.11.5",
1212
"psutil==6.*",
1313
]
1414
readme = "README.md"

rlbot/managers/rendering.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ def begin_rendering(self, group_id: str = DEFAULT_GROUP_ID):
8484
"""
8585
Begins a new render group. All render messages added after this call will be part of this group.
8686
"""
87-
if self._group_id is not None:
87+
if self.is_rendering():
8888
self._logger.error(
8989
"begin_rendering was called twice without end_rendering."
9090
)
@@ -144,6 +144,13 @@ def draw(
144144
| flat.Rect3D
145145
),
146146
):
147+
if not self.is_rendering():
148+
self._logger.warning(
149+
"Attempted to draw without a render group."
150+
"Please call `begin_rendering` first, and then `end_rendering` after."
151+
)
152+
return
153+
147154
self._current_renders.append(flat.RenderMessage(render))
148155

149156
def draw_line_3d(

rlbot/version.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
__version__ = "5.0.0-beta.13"
1+
__version__ = "5.0.0-beta.14"
22

33

44
RESET_SEQ = "\033[0m"
@@ -15,6 +15,9 @@ def _get_color(color: int) -> str:
1515
)
1616

1717
RELEASE_NOTES = {
18+
"5.0.0-beta.14": """
19+
- Add warning to Renderer when trying to render without calling `begin_rendering` first
20+
""",
1821
"5.0.0-beta.13": """
1922
- Account for partial reads in `SocketRelay`
2023
- Fix managers not ensuring they have received all prerequisite data before initializing

0 commit comments

Comments
 (0)