[ENHANCEMENT]: Consider removing the use of `argument_type` and `result_type` from `hasher` in cuco `default_filter_policy`
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 35/100
Rechercherichtung
Durchsuche cuCollections nach hasher::argument_type und hasher::result_type und untersuche anschließend deren Verwendung in Bloom-Filter-Richtlinien. Erfasse die betroffenen Hasher und Richtlinien, bevor du irgendetwas änderst. Als erledigt gilt die Aufgabe, wenn die veralteten Aliase und ihre Verwendungen in cuCollections entfernt sind und die relevanten Builds und Tests weiterhin erfolgreich sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Is your feature request related to a problem? Please describe.
The member type std::hash::argument_type is deprecated in C++17 and removed in C++20, so we should consider syncing with STL and remove them from our hashers as well as their use in bloom filter policies.
Ref: https://en.cppreference.com/w/cpp/utility/hash
Originally posted by @bdice in https://github.com/rapidsai/cudf/pull/17289#discussion_r1876965059
Describe the solution you'd like
Consider refactoring to not use hasher::argument_type and hasher::result_type across cuCollections.
Describe alternatives you've considered
Keep using the deprecated hasher type aliases until C++20
Additional context
No response
- Vorherrschende Sprache
- Cuda
- Sterne
- 674
- Forks
- 123
- Ø Merge
- 1 T. 13 Std.
- Gemergte PRs (30 T.)
- 8
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus NVIDIA/cuCollections
-
Add cuco::detail::stream_sync(cuda::stream_ref) to centralize CCCL version-specific API namingEvtl. wieder frei @0z5a hat das vor 40 Tagen übernommen, und es ist kein Pull Request offen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
NVIDIA/cuCollections#840 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
nvidia-runners
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 25/100
NVIDIA/cuCollections#853 ·
Maintainer antworten meist innerhalb von 1 Tag
-
topic: performance type: feature request
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
NVIDIA/cuCollections#817 · 7 Kommentare · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
good first issue P2: Nice to have type: improvement
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 38/100
NVIDIA/cuCollections#805 · 4 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
[FEA] Add MPSC/MPMC concurrent queueEvtl. wieder frei @sleeepyjack hat das vor 279 Tagen übernommen, und es ist kein Pull Request offen. Offenmaintainer-only type: feature request
NVIDIA/cuCollections#791 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in NVIDIA/cuCollections
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
nebari-dev/nebi#630 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
router-for-me/CLIProxyAPI#7084 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 83/100
speakeasy-api/openapi#284 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
jwage/phpamqplib-messenger#159 ·