Skip to content

Conversation

@simonbyrne
Copy link
Collaborator

@simonbyrne simonbyrne commented Jul 25, 2025

This should speed up development, as you can now build and run a single test:

make test_00_operators_interp_test
test/test_00_operators_interp_test

make test still builds and runs all tests (using ctest to run in parallel).

@cliffburdick I'm not sure if this is what you had in mind for #229?

@copy-pr-bot
Copy link

copy-pr-bot bot commented Jul 25, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@cliffburdick
Copy link
Collaborator

/build

1 similar comment
@cliffburdick
Copy link
Collaborator

/build

@simonbyrne simonbyrne force-pushed the sbyrne/test-executables branch from 9db80d0 to d80d380 Compare July 28, 2025 17:09
@simonbyrne
Copy link
Collaborator Author

@cliffburdick it looks like CI was directly calling test/matx_test. I added a script to provide a compatible entry point to the tests.

@cliffburdick
Copy link
Collaborator

/build

1 similar comment
@cliffburdick
Copy link
Collaborator

/build

@cliffburdick
Copy link
Collaborator

Looks like it doesn't like some of the docstring updates:

/home/jenkins/workspace/unit-tests/docs_input/api/logic/truth/allclose.rst:10: WARNING: doxygenfunction: Unable to resolve function "allclose" with arguments (OutType, const InType1&, const InType2&, double, double, [[maybe_unused]] const HostExecutor<MODE>&) in doxygen xml output for project "MatX" from directory: /home/jenkins/workspace/unit-tests/build/docs_input/doxygen/xml.
Potential matches:
- template<typename OutType, typename InType1, typename InType2, ThreadsMode MODE> void __MATX_INLINE__ allclose(OutType dest, const InType1 &in1, const InType2 &in2, double rtol, double atol, const HostExecutor<MODE> &exec)
- template<typename OutType, typename InType1, typename InType2> void __MATX_INLINE__ allclose(OutType dest, const InType1 &in1, const InType2 &in2, double rtol, double atol, cudaExecutor exec = 0) [docutils]
/home/jenkins/workspace/unit-tests/docs_input/api/stats/corr/corr.rst:8: WARNING: doxygenfunction: Unable to resolve function "corr" with arguments (const In1Type&, const In2Type&, matxConvCorrMode_t, [[maybe_unused]] matxConvCorrMethod_t) in doxygen xml output for project "MatX" from directory: /home/jenkins/workspace/unit-tests/build/docs_input/doxygen/xml.
Potential matches:
- template<typename In1Type, typename In2Type> __MATX_INLINE__ auto corr(const In1Type &i1, const In2Type &i2, const int32_t (&axis)[1], matxConvCorrMode_t mode, matxConvCorrMethod_t method)
- template<typename In1Type, typename In2Type> __MATX_INLINE__ auto corr(const In1Type &i1, const In2Type &i2, matxConvCorrMode_t mode, matxConvCorrMethod_t method) [docutils]
/home/jenkins/workspace/unit-tests/docs_input/api/stats/corr/corr.rst:9: WARNING: doxygenfunction: Unable to resolve function "corr" with arguments (const In1Type&, const In2Type&, const int32_t (&)[1], matxConvCorrMode_t, [[maybe_unused]] matxConvCorrMethod_t) in doxygen xml output for project "MatX" from directory: /home/jenkins/workspace/unit-tests/build/docs_input/doxygen/xml.
Potential matches:
- template<typename In1Type, typename In2Type> __MATX_INLINE__ auto corr(const In1Type &i1, const In2Type &i2, const int32_t (&axis)[1], matxConvCorrMode_t mode, matxConvCorrMethod_t method)
- template<typename In1Type, typename In2Type> __MATX_INLINE__ auto corr(const In1Type &i1, const In2Type &i2, matxConvCorrMode_t mode, matxConvCorrMethod_t method) [docutils]
- ```

@simonbyrne
Copy link
Collaborator Author

Let me revert that: it wasn't working locally for me without those.

@simonbyrne simonbyrne force-pushed the sbyrne/test-executables branch from d80d380 to ede7102 Compare July 28, 2025 20:32
@cliffburdick
Copy link
Collaborator

/build

1 similar comment
@cliffburdick
Copy link
Collaborator

/build

@cliffburdick cliffburdick merged commit df8c5b2 into NVIDIA:main Jul 29, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants