john
/
McRogueFace
1
0
Fork 0
Code Issues 59 Pull Requests Projects 1 Releases Wiki Activity

Error message on grid.at does not explain which value was invalid well #77

New Issue
Open
opened 2025-03-05 20:26:55 +00:00 by john · 0 comments
john commented 2025-03-05 20:26:55 +00:00
Owner
Copy Link

image

  File "/home/john/Development/McRogueFace/build/scripts/cos_level.py", line 178, in generate
    self.grid.at((tx, y)).walkable = True
    ^^^^^^^^^^^^^^^^^^^^^
ValueError: x value out of range (0, Grid.grid_y)

ValueError says x, but the range says (0, Grid.grid_y). Probably a copy/paste error. It would be nice to see the value given here as well

![image](/attachments/f1996e60-0fcd-4343-9d80-21241e819782) ``` File "/home/john/Development/McRogueFace/build/scripts/cos_level.py", line 178, in generate self.grid.at((tx, y)).walkable = True ^^^^^^^^^^^^^^^^^^^^^ ValueError: x value out of range (0, Grid.grid_y) ``` ValueError says `x`, but the range says `(0, Grid.grid_y)`. Probably a copy/paste error. It would be nice to see the value given here as well
image.png
72 KiB
john added the
Bugfix
 label 2025-03-05 20:27:01 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
master
grid_entity_integration
reprs_and_member_names
break_up_ui_h
standardize_font_handling
standardize_vector_handling
standardize_color_handling
standardize_texture_handling
raii_pyobjects
rebase-for-7DRL_Feb-2024
engjam_uicollection
engjam_ui_overhaul
Labels
Clear labels   
Alpha Release Requirement

This issue is a blocker to 0.1 Alpha release of McRogueFace.

  
Bugfix

   
Demo Target

Functionality to demonstrate; Depends on new features, but this issue isn't for writing them.

  
Documentation

   
Major Feature

Significant time and effort required.

  
Minor Feature

Some effort required to create or overhaul functionality.

  
Refactoring & Cleanup

no new functionality, just improving the codebase.

  
Tiny Feature

quick and easy - a few lines or with little interconnection around the codebase

No Label
Alpha Release Requirement
Bugfix
Demo Target
Documentation
Major Feature
Minor Feature
Refactoring & Cleanup
Tiny Feature
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: john/McRogueFace#77
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?