Have `paduapts` return the data as the transpose of the current format?

Offen
  • Stimmt etwas nicht?
#12 8 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
45/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
julia
Bereich
data

Rechercherichtung

Beginne bei der Implementierung von paduapts(n) und untersuche die Aufrufer, die sich auf die aktuelle Form N×2 stützen. Bestätige, dass die vorgeschlagene Rückgabeform 2×N die Punktdaten beibehält, und aktualisiere alle betroffenen Erwartungen oder Tests; abgeschlossen ist die Aufgabe, wenn paduapts das neue Format ohne unnötige Allokation zurückgibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

question

Right now paduapts(n) returns an N x 2 matrix. I'm proposing that it instead return a 2 x N matrix. The reason is that this allows for a no-op conversion to a Vector{Vec{2,Float64}}: the current code works

pts=paduapts(10)
ptr=reinterpret(Ptr{Vec{2,Float64}},pointer(pts'))  # reinterpret the memory of pts'
unsafe_wrap(Vector{Vec{2,Float64}},
            ptr,
            size(pts,1),true)

With the proposed change, the pts' would become pts and no new memory would need to be allocated.

Vorherrschende Sprache
Julia
Sterne
284
Forks
28
Ø Merge
17 Std. 49 Min.
Gemergte PRs (30 T.)
1

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus JuliaApproximation/FastTransforms.jl

Alle Issues in JuliaApproximation/FastTransforms.jl

Ähnliche Issues

Weitere Issues zu Julia

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.