`@functor` is confused about properties vs fields
Una pull request collegata è già stata integrata.
- #48 di @mcabbott — integrata
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- julia
- Ambito
- backend-api-design
Direzione di ricerca
Inizia dalle righe 11-16 di src/functor.jl, che usano fieldnames e getproperty. Confronta quella ricerca con il conflitto di ProtoStruct.jl dell'issue, quindi aggiorna in modo coerente l'accesso ai campi per usare getfield; il lavoro è completo quando Functors non dipende più dal comportamento sovraccaricato delle proprietà.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
It was noticed in https://github.com/FluxML/Flux.jl/issues/2107 that Functors.jl + ProtoStruct.jl doesn't work, as Functors uses fieldnames + getproperty, which is overloaded by ProtoStruct.jl.
https://github.com/FluxML/Functors.jl/blob/v0.3.0/src/functor.jl#L11-L16
This is a bug, Functors should use getfield to be consistent. (Possibly the code was written before getproperty existed?)
- Lingua principale
- Julia
- Stelle
- 123
- Fork
- 18
- Merge medio
- 11h 21m
- PR unite (30g)
- 2
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 FluxML/Functors.jl
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
FluxML/Functors.jl#111 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
FluxML/Functors.jl#81 ·
-
breaking
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
FluxML/Functors.jl#77 · 1 commento · 1 reazione ·
-
Taggable functorsAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
FluxML/Functors.jl#41 · 4 commenti · 1 reazione ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
FluxML/Functors.jl#40 · 5 commenti ·
Tutte le issue di FluxML/Functors.jl
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
MagneticResonanceImaging/MRIReco.jl#366 ·
I maintainer di solito rispondono entro 3 giorni
-
`AdamW` gives `NaN` for zero-gradient entries of `Float16` parameters (epsilon underflows to 0)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
FluxML/Optimisers.jl#253 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
JuliaGraphs/GraphNeuralNetworks.jl#736 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
`poissonians` is not public, unlike `brownians` and `jumps`Forse già presa @isaacsas l’ha presa oggi. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 86/100
SciML/ModelingToolkit.jl#5389 ·
I maintainer di solito rispondono entro 2 giorni