Python OPI checker: add check for stray widgets

Open Beginner friendly
#2,078 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
75/100
Issue type
Feature
Clarity
Clearly specified
Activity status
Stale
Tech stack
python
Domain
tooling

Research direction

Open check_opi_format.py and find the existing OPI validation checks. Add validation for widget x and y positions using the proposed limits, then verify that the stray values from issue #1987 are reported while positions within the limits pass.

Written by the indexing model from the issue text.

Description

Following an issue in #1987 where we had a stray widget located at

<x>-2147483647</x>
<y>2147483496</y>

it would save debugging time later if the script checked OPIs for strange widget positions.

The check_opi_format.py script should be modified to check for these values. The following limits might be good starting points (we may need to make adjustments to these later):

-10 < x < 1000
-10 < y < 1000
Dominant language
No language data
Stars
6
Forks
2
Avg merge
16h 40m
Merged PRs (30d)
2

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from ISISComputingGroup/IBEX

All issues in ISISComputingGroup/IBEX

Similar issues

More DevTools issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.