Skip to content

Commit

Permalink
remoce user_stream and device_id arguments
Browse files Browse the repository at this point in the history
  • Loading branch information
madsbk committed Feb 27, 2025
1 parent da5f8d8 commit 2a8f203
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 32 deletions.
25 changes: 6 additions & 19 deletions python/kvikio/kvikio/_lib/libnvcomp.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ from kvikio._lib.nvcomp_cxx_api cimport (
LZ4Manager,
SnappyManager,
create_manager,
cudaStream_t,
nvcompBatchedANSDefaultOpts,
nvcompBatchedANSOpts_t,
nvcompBatchedBitcompFormatOpts,
Expand Down Expand Up @@ -156,9 +155,7 @@ cdef class _ANSManager(_nvcompManager):
):
self._impl = <nvcompManagerBase*>new ANSManager(
uncomp_chunk_size,
<nvcompBatchedANSOpts_t>nvcompBatchedANSDefaultOpts, # TODO
<cudaStream_t><void*>0, # TODO
device_id
<nvcompBatchedANSOpts_t>nvcompBatchedANSDefaultOpts
)


Expand All @@ -174,9 +171,7 @@ cdef class _BitcompManager(_nvcompManager):
cdef opts = nvcompBatchedBitcompFormatOpts(bitcomp_algo, data_type)
self._impl = <nvcompManagerBase*>new BitcompManager(
uncomp_chunk_size,
opts,
<cudaStream_t><void*>0, # TODO
device_id
opts
)


Expand All @@ -189,9 +184,7 @@ cdef class _CascadedManager(_nvcompManager):
):
self._impl = <nvcompManagerBase*>new CascadedManager(
_options["chunk_size"],
<nvcompBatchedCascadedOpts_t>nvcompBatchedCascadedDefaultOpts, # TODO
<cudaStream_t><void*>0, # TODO
device_id,
<nvcompBatchedCascadedOpts_t>nvcompBatchedCascadedDefaultOpts
)


Expand All @@ -206,9 +199,7 @@ cdef class _GdeflateManager(_nvcompManager):
cdef opts = nvcompBatchedGdeflateOpts_t(algo)
self._impl = <nvcompManagerBase*>new GdeflateManager(
uncomp_chunk_size,
opts,
<cudaStream_t><void*>0, # TODO
device_id
opts
)


Expand All @@ -227,9 +218,7 @@ cdef class _LZ4Manager(_nvcompManager):
cdef opts = nvcompBatchedLZ4Opts_t(data_type)
self._impl = <nvcompManagerBase*>new LZ4Manager(
uncomp_chunk_size,
opts,
<cudaStream_t><void*>0, # TODO
device_id
opts
)


Expand All @@ -245,9 +234,7 @@ cdef class _SnappyManager(_nvcompManager):
# everything else works.
self._impl = <nvcompManagerBase*>new SnappyManager(
uncomp_chunk_size,
<nvcompBatchedSnappyOpts_t>nvcompBatchedSnappyDefaultOpts,
<cudaStream_t><void*>0, # TODO
device_id
<nvcompBatchedSnappyOpts_t>nvcompBatchedSnappyDefaultOpts
)


Expand Down
14 changes: 1 addition & 13 deletions python/kvikio/kvikio/_lib/nvcomp_cxx_api.pxd
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Copyright (c) 2022 Carson Swope
# Use, modification, and distribution is subject to the MIT License
# https://github.com/carsonswope/py-nvcomp/blob/main/LICENSE)
# SPDX-FileCopyrightText: Copyright (c) 2022-2023, NVIDIA CORPORATION & AFFILIATES.
# SPDX-FileCopyrightText: Copyright (c) 2022-2025, NVIDIA CORPORATION & AFFILIATES.
# All rights reserved.
# SPDX-License-Identifier: MIT
# Permission is hereby granted, free of charge, to any person obtaining a
Expand Down Expand Up @@ -139,8 +139,6 @@ cdef extern from "nvcomp/ans.hpp":
ANSManager(
size_t uncomp_chunk_size,
const nvcompBatchedANSOpts_t& format_opts,
cudaStream_t user_stream,
const int device_id
) except +

# C++ Concrete Bitcomp Manager
Expand All @@ -155,8 +153,6 @@ cdef extern from "nvcomp/bitcomp.hpp":
BitcompManager(
size_t uncomp_chunk_size,
const nvcompBatchedBitcompFormatOpts& format_opts,
cudaStream_t user_stream,
const int device_id
) except +

# C++ Concrete Cascaded Manager
Expand All @@ -174,8 +170,6 @@ cdef extern from "nvcomp/cascaded.hpp" nogil:
CascadedManager(
size_t uncomp_chunk_size,
const nvcompBatchedCascadedOpts_t& options,
cudaStream_t user_stream,
int device_id
)

# C++ Concrete Gdeflate Manager
Expand All @@ -189,8 +183,6 @@ cdef extern from "nvcomp/gdeflate.hpp":
GdeflateManager(
int uncomp_chunk_size,
const nvcompBatchedGdeflateOpts_t& format_opts,
cudaStream_t user_stream,
const int device_id
) except +

# C++ Concrete LZ4 Manager
Expand All @@ -204,8 +196,6 @@ cdef extern from "nvcomp/lz4.hpp":
LZ4Manager(
size_t uncomp_chunk_size,
const nvcompBatchedLZ4Opts_t& format_opts,
cudaStream_t user_stream,
const int device_id
) except +

# C++ Concrete Snappy Manager
Expand All @@ -219,6 +209,4 @@ cdef extern from "nvcomp/snappy.hpp":
SnappyManager(
size_t uncomp_chunk_size,
const nvcompBatchedSnappyOpts_t& format_opts,
cudaStream_t user_stream,
const int device_id
) except +

0 comments on commit 2a8f203

Please sign in to comment.