Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Support any hash constructor in hash_test.cu::check_hash_result

Aperta
  • Qualcosa non va?
#520 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
35/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
cpp
Ambito
testing

Direzione di ricerca

Apri hash_test.cu e individua check_hash_result e ogni punto in cui viene chiamato nel file. Esamina l’interfaccia proposta del costruttore variadico, sposta ogni argomento seed alla fine ed esegui i test di hash per confermare che i risultati esistenti continuino a passare e che la costruzione funzioni senza un seed.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

good first issue type: improvement 
          I see why you made this: Because we have hash functions that don't take a seed.  However, we'll eventually have the `SigBitHash` that takes not a seed but rather a list or a range or whatever.

I think that it would be better to not write a million of these functions. The right way to do it is with a variadic template. Also, you could make that a different PR. The code would look like this:

template <typename Hash, typename... HashConstructorArgs>
static __host__ __device__ bool check_hash_result(typename Hash::argument_type const& key,
                                                  typename Hash::result_type expected,
                                                  HashConstructorArgs&&... hash_constructor_args) noexcept
{
  Hash h(std::forward<HashConstructorArgs>(hash_constructor_args)...);
  return (h(key) == expected);
}

The seed is now the last argument so you need to find all occurrences of check_hash_result in this file and make the seed the last argument.

After doing this change, the function will now be compatible with all hash construction.

If you want to read more about std::forward, these are good:

The short of it is that std::forward written this way with a "universal reference" makes it so that this code works well when:

  • you pass in something that has a name, like check_hash_result(my_key, my_expected, my_seed). (my_seed has a "name", it's a left-value aka l-value aka something that can be on the left side of an assignment)
  • you pass in something nameless, like check_hash_result(my_key, my_expected, 12345). (12345 has no "name", it's a right-value aka r-value aka something that can not be on the left side of an assignment statement)

When used like this, the compiler will correctly optimize if the value of the seed or whatever is passed in there needs to be copied or moved.

Originally posted by @esoha-nvidia in https://github.com/NVIDIA/cuCollections/pull/514#discussion_r1653534402

Lingua principale
Cuda
Stelle
671
Fork
122
Merge medio
4g 19h
PR unite (30g)
10

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di NVIDIA/cuCollections

Tutte le issue di NVIDIA/cuCollections

Issue simili

Altre issue su Testing & QA

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.