signature tests being misled by **kwargs

Open

#171 opened on Feb 24, 2023 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 2/5
Estimated time: 1-3 hours
Newbie friendliness: 42/100

Issue type: Bug
Clarity: Clearly specified
Activity status: Stale
Tech stack: python
Domain: testing

Research direction

Start in array_api_tests/test_signatures.py around line 92, where the meshgrid signature is checked. Run the relevant signature test and compare its handling of explicit parameters with **kwargs; done means cupy.meshgrid's indexing keyword is not rejected solely because its signature exposes kwargs.

Written by the indexing model from the issue text.

Description

medium priority

The meshgrid signature test fails with cupy:

E           AssertionError: Argument 'indexing' missing from signature
E           assert 'indexing' in odict_keys(['xi', 'kwargs'])
E            +  where 'indexing' = <Parameter "indexing: str = 'xy'">.name
E            +  and   odict_keys(['xi', 'kwargs']) = <built-in method keys of mappingproxy object at 0x7fc946212170>()
E            +    where <built-in method keys of mappingproxy object at 0x7fc946212170> = mappingproxy(OrderedDict([('xi', <Parameter "*xi">), ('kwargs', <Parameter "**kwargs">)])).keys
E            +      where mappingproxy(OrderedDict([('xi', <Parameter "*xi">), ('kwargs', <Parameter "**kwargs">)])) = <Signature (*xi, **kwargs)>.parameters

array_api_tests/test_signatures.py:92: AssertionError

cupy.meshgrid does have the indexing keyword, but its signature is meshgrid(*x, **kwargs). If we can't check a keyword argument directly, we should assume it is present with **kwargs, and let the individual tests suss out whether it really supports it.

Dominant language: Python
Stars: 74
Forks: 54
Avg merge: 4h 35m
Merged PRs (30d): 4

Contributor guide

No contributing guide indexed for this repository

First steps

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

Similar issues

issue: imported chats all show as unread in the sidebar Open

bug confirmed issue

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

open-webui/open-webui#30750 · 1 comment · Opened Sep 24, 2026
OGS AmBe example has no source.timing block, so every event sits at time zero Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

lanl/ScintiPix#61 · Opened Sep 24, 2026
Make Liquid Glass the default theme for the Research build Open

enhancement

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

OpenwaterHealth/openmotion-bloodflow-app#604 · 1 comment · Opened Sep 24, 2026
tests/puppeteer/test_openai_api_key_ui.js targets controls that do not exist and is run by no workflow Open

Difficulty 2/5 1-3 hours Newbie friendliness 70/100

LearningCircuit/local-deep-research#6766 · Opened Sep 23, 2026
Delete unused args argument from main() in Python files Open

good first issue

Difficulty 1/5 Under an hour Newbie friendliness 90/100

ros2/launch#1014 · Opened Sep 23, 2026

More Python issues