Skip to content

Update CtypesLinker::add_cu error message to include fp16 usage #8744

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sklam merged 14 commits into from
Mar 28, 2023
Merged

Update CtypesLinker::add_cu error message to include fp16 usage #8744

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
aec2c5d
Update error message to include f16 usage
testhound Feb 8, 2023
12288b8
Fix flake8 error
testhound Feb 8, 2023
369b016
Merge branch 'main' into testhound/cu_linking_err_msg
testhound Feb 13, 2023
7704080
Update error message to be more informative
testhound Feb 15, 2023
db210b6
Separate out float16 error exception from cu linking
testhound Feb 16, 2023
275243e
Run test case unconditionally, fix file not being found and override …
testhound Feb 16, 2023
5212ac6
Add new test to verify float16 exception
testhound Feb 28, 2023
467daf6
Merge branch 'main' into testhound/cu_linking_err_msg
testhound Feb 28, 2023
dad4421
Fix typo
testhound Feb 28, 2023
6587af4
Fix typo and simplify test case
testhound Feb 28, 2023
932046d
Skip test if minor version compatability used
testhound Feb 28, 2023
4e4ecd8
Merge branch 'main' into testhound/cu_linking_err_msg
testhound Mar 17, 2023
51f914d
Merge remote-tracking branch 'numba/main' into testhound/cu_linking_e…
gmarkall Mar 28, 2023
18dae59
CUDA test_linker.py: fix jitlink.cu path
gmarkall Mar 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Update error message to include f16 usage
  • Loading branch information
@testhound
testhound committed Feb 8, 2023
commit aec2c5d6ed1afcb862ce883753aa4020772641fd
7 changes: 5 additions & 2 deletions numba/cuda/cudadrv/driver.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2721,8 +2721,11 @@ def add_file(self, path, kind):
raise LinkerError(msg)

def add_cu(self, path, name):
raise NotImplementedError("Linking CUDA source files is not supported "
"with the ctypes binding. ")
raise NotImplementedError("Support for fp16 or linking CUDA source files "
"requires the use of the Nvidia CUDA "
"bindings and enabling the "
"NUMBA_CUDA_USE_NVIDIA_BINDING environment variable. "
"See documentation for me details")

def complete(self):
cubin = c_void_p(0)
Expand Down
4 changes: 2 additions & 2 deletions numba/cuda/tests/cudadrv/test_linker.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -191,8 +191,8 @@ def kernel(x):

@skip_with_cuda_python
def test_linking_cu_ctypes_unsupported(self):
msg = ('Linking CUDA source files is not supported with the ctypes '
'binding')
msg = ('Support for fp16 or linking CUDA source files '

@gmarkall gmarkall Feb 16, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The expected message doesn't seem to match the message added in dispatcher.py. Shouldn't this test be failing?

@testhound testhound Feb 16, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gmarkall the test was not failing it appears because it was not being run due to the skip_with_cuda_python decorator.

'requires the use of the Nvidia CUDA bindings')
with self.assertRaisesRegex(NotImplementedError, msg):
@cuda.jit('void()', link=['jitlink.cu'])
def f():
Expand Down