`MVector` type yields `ImmutableDiffResult`
Una pull request collegata è già stata integrata.
- #18 di @charleskawczynski — integrata
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia con src/DiffResults.jl alle righe 31-52 e riproduci il comportamento segnalato usando un MVector o MArray, come descritto in LsqFit.jl e StaticArrays.jl. Verifica se la pull request già integrata ha modificato questo dispatch, quindi verifica che l’input mutabile segnalato produca un MutableDiffResult senza influire sui casi esistenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Hello, I'm using this package through LsqFit.jl. I have a 3d parameter vector, and would like to speed up the code by leveraging StaticArrays.jl, which basically come for free by passing a StaticVector instead of a Vector. In order to support in-place operations, we can further pass an MVector, i.e. a modifiable statically sized array. However, as seen below, Static Arrays always lead to ImmutableDiffResults.
I've done the following overloads to make it work for my case:
DiffResult(value::MArray, derivs::Tuple{Vararg{MArray}}) = MutableDiffResult(value, derivs)
DiffResult(value::Union{Number, AbstractArray}, derivs::Tuple{Vararg{MVector}}) = MutableDiffResult(value, derivs)
I think this is a useful feature and currently can be considered a bug. If there's interest in merging this I can also open a PR.
- Lingua principale
- Julia
- Stelle
- 37
- Fork
- 9
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di JuliaDiff/DiffResults.jl
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
JuliaDiff/DiffResults.jl#27 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
JuliaDiff/DiffResults.jl#26 · 4 commenti ·
-
Document why re-aliasing is importantForse già presa @devmotion l’ha presa 430 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
JuliaDiff/DiffResults.jl#17 · 1 commento · 3 reazioni ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
JuliaDiff/DiffResults.jl#10 · 2 commenti ·
-
confused about interfaceForse già presa @devmotion l’ha presa 430 giorni fa. Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
JuliaDiff/DiffResults.jl#4 · 3 commenti ·
Tutte le issue di JuliaDiff/DiffResults.jl
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
I maintainer di solito rispondono entro 1 giorno
-
lux
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
JuliaGraphs/GraphNeuralNetworks.jl#732 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
erfinv(::Float16) and erfcinv(::Float16) are wrong for abs(x) <= 0.75 (erfinv(Float16(0)) == 0.2201)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
JuliaMath/SpecialFunctions.jl#571 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100