lanl/singularity-eos

Should we remove forwarding references from PTE solvers?

Offen

#438 geöffnet am 29.11.2024

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (21 Forks)auto 404
bugclean-updiscussionhelp wantedinterfacequestion

Repository-Metriken

Stars
 (36 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

The segfault on HIP was caused by an access-out-of-scope error caused by passing pointers by reference into the PTE solvers. See #437 . Perhaps we should consider removing the forwarding references and pass only by value into the PTE solvers (with the exception of PTE Solver Base, which requires pass-by-reference as it is used as the first part of the derived class constructors).

Ping @jhp-lanl @jdolence @dholladay00 @mauneyc-LANL and @rbberger for feedback

Contributor Guide

Research-Richtung
Lesen Sie Issue #437, um den durch die Übergabe von Zeigern als Referenz verursachten Segfault zu verstehen. Untersuchen Sie dann den PTE Solver Code und konzentrieren Sie sich darauf, wie Weiterleitungsreferenzen verwendet werden. Planen Sie, nur die PTE Solver zu ändern, um per Wert zu übergeben, mit Ausnahme der PTE Solver Base. Testen Sie die Änderungen auf HIP, um sicherzustellen, dass kein Segfault auftritt.
Tech Stack
cpp
Domain
hpc
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40