FluxML/Flux.jl

Provide aliases for unicode fields and keywords

Open

#503 aperta il 21 nov 2018

Vedi su GitHub
 (5 commenti) (0 reazioni) (0 assegnatari)Julia (619 fork)batch import
good first issuehelp wanted

Metriche repository

Star
 (4725 star)
Metriche merge PR
 (Merge medio 4h 27m) (2 PR mergiate in 30 g)

Descrizione

I'm thinking of adding aliases for keyword arguments and structure fields that currently require using unicode characters to access. Examples include the σ in the Conv field names, and almost all of the fields in BatchNorm:

julia> fieldnames(BatchNorm)
(:λ, :β, :γ, :μ, :σ, :ϵ, :momentum, :active)

I only have three data points so far, but when other people start to use Flux this has been something that bothers them.

Questions:

  • Is this something we should do?
  • Would you prefer having ascii names added as aliases, or replacing the current instances of unicode with ascii names and then adding the current unicode names back as aliases?
  • Should the ascii name simply spell out the unicode symbol, or describe meaning of the field/variable? For example, should the alias for σ in Conv be activation or sigma? (My preference would be activation.)

Guida contributor

Direzione di ricerca
Ispeziona i nomi dei campi attuali nei livelli BatchNorm e Conv, quindi aggiungi alias ASCII seguendo le convenzioni di denominazione della libreria. Garantisci la compatibilità all'indietro mantenendo i nomi Unicode.
Tech stack
Nessuno
Dominio
ai
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JuliaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75