09Catho opened a new pull request, #55483:
URL: https://github.com/apache/arrow/pull/55483

   ### Rationale for this change
   
   Four tests use NumPy but are missing `pytest.mark.numpy`. Without that 
marker, the no-NumPy job does not exercise them and the tests can fail with an 
AttributeError when an off-by-default group runs without NumPy.
   
   ### What changes are included in this PR?
   
   Add `pytest.mark.numpy` to the three large-memory conversion tests and the 
CUDA-backed `test_non_cpu_array` test named in #55458.
   
   ### Are these changes tested?
   
   - `python -m py_compile python/pyarrow/tests/test_convert_builtin.py 
python/pyarrow/tests/test_array.py` passed.
   - An AST check confirmed all four target functions have the NumPy marker.
   - `git diff --check` passed.
   - The focused pytest commands could not collect because this shallow source 
checkout does not contain the generated `pyarrow.lib` extension. No pytest 
result is claimed.
   
   ### Are there any user-facing changes?
   
   No. This only corrects test selection for environments without NumPy.
   
   ### Was AI used for this PR?
   
   In accordance to the [AI generation 
guidelines](https://arrow.apache.org/docs/dev/developers/overview.html#ai-generated-code),
 please disclose below whether and how AI was used in this PR.
   
   **PR code and description written by:**
   
   - [ ] Human
   - [x] AI
   
   **Reviewed before submission by:**
   
   - [ ] Human
   - [x] AI
   - [ ] Not reviewed
   
   * GitHub Issue: #55458


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

  • [PR] GH-55458: [Python] Mark N... 09Catho (via GitHub) via github

Reply via email to