Skip to content
This repository was archived by the owner on Aug 1, 2022. It is now read-only.

Raise a more descriptive error message by reporting the erroneous box name and available box names. #59

Open
BuysDB wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Raise a more descriptive error message by reporting the erroneous box name and available box names. #59

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c0baa2d
Raise a more descriptive error message by reporting the erroneous box…
BuysDB Oct 7, 2020
299e74d
Reduced linecount
BuysDB Oct 7, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions statannot/statannot.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -468,10 +468,10 @@ def get_box_data(box_plotter, boxName):
# Build the list of box data structure pairs
box_struct_pairs = []
for i_box_pair, (box1, box2) in enumerate(box_pairs):
valid = box1 in box_names and box2 in box_names
if not valid:
raise ValueError("box_pairs contains an invalid box pair.")
pass

if box1 not in box_names or box2 not in box_names:
raise ValueError(f"box_pairs contains an invalid box pair, expected {box1 if not box1 in box_names else box2} in {box_names}")

# i_box_pair will keep track of the original order of the box pairs.
box_struct1 = dict(box_structs_dic[box1], i_box_pair=i_box_pair)
box_struct2 = dict(box_structs_dic[box2], i_box_pair=i_box_pair)
Expand Down