KhronosGroup / KhronosGroup/WebGL

Resolve issues with illegal constructs in dEQP shader tests

Open
#885 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
HTML
Stars
2.9k
Forks
704
Avg merge
2d 9h
Merged PRs (30d)
4

Description

In pull request #879 there is a currently-ongoing discussion about the modification and removal of some recently-added tests from drawElements' test suite.

Some of the changes that were made in that pull request should be revisited. The modifications that were made seem to have minimized the utility of the tests.

Contributor guide

Open the contributing guide

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.

Research direction

Start with pull request #879 and its discussion about the recently added dEQP shader tests. Identify which illegal constructs and test modifications are disputed, then trace the affected tests in drawElements' test suite. The work is done when the intended test coverage is agreed and the affected tests no longer rely on inappropriate constructs.

Written by the indexing model from the issue text.

Assessment

Domain
testing
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.