[ENHANCEMENT]: Consider removing the use of `argument_type` and `result_type` from `hasher` in cuco `default_filter_policy`
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 35/100
Direção de pesquisa
Pesquise por hasher::argument_type e hasher::result_type em cuCollections e, em seguida, inspecione seu uso nas políticas de filtros Bloom. Mapeie os hashers e as políticas afetados antes de alterar qualquer coisa. Considera-se concluído quando os aliases obsoletos e seus usos forem removidos em cuCollections e os builds e testes relevantes continuarem sendo bem-sucedidos.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- Cuda
- Estrelas
- 674
- Forks
- 123
- Merge médio
- 1d 13h
- PRs com merge (30d)
- 8
Preparar o ambiente
Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de NVIDIA/cuCollections
-
Add cuco::detail::stream_sync(cuda::stream_ref) to centralize CCCL version-specific API namingTalvez livre de novo @0z5a assumiu há 41 dias e não há nenhum pull request aberto. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
NVIDIA/cuCollections#840 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
nvidia-runners
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 25/100
NVIDIA/cuCollections#853 ·
Mantenedores costumam responder em até 1 dia
-
topic: performance type: feature request
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
NVIDIA/cuCollections#817 · 7 comentários · 1 reação ·
Mantenedores costumam responder em até 1 dia
-
good first issue P2: Nice to have type: improvement
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 38/100
NVIDIA/cuCollections#805 · 4 comentários ·
Mantenedores costumam responder em até 1 dia
-
[FEA] Add MPSC/MPMC concurrent queueTalvez livre de novo @sleeepyjack assumiu há 280 dias e não há nenhum pull request aberto. Abertamaintainer-only type: feature request
NVIDIA/cuCollections#791 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
Todas as issues de NVIDIA/cuCollections
Issues semelhantes
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 84/100
wp-graphql/wp-graphql-woocommerce#1026 ·
-
bug integrator
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
grafana/sigma-rule-deployment#431 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
datamodel-code-generator/datamodel-code-generator#5587 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
Shell: serve the home app's Cookie Policy on a hard reloadTalvez já em andamento @2001anshkaushik assumiu hoje. Abertafeature module:ai
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
rocketride-org/rocketride-server#2899 · 1 comentário ·
Mantenedores costumam responder em até 2 dias