Merge on DefaultOrderedDicts should call the ordered merge function
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 42/100
Línea de trabajo
Comienza reproduciendo el problema con los dos ejemplos de DefaultOrderedDict e inspeccionando el dispatch de merge mostrado por @which. Compáralo con la implementación de merge de OrderedDict en ordered_dict.jl; se considera terminado cuando al hacer merge de DefaultOrderedDicts se conserva su orden en lugar de usar el merge de dict normal.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
julia> d = DefaultOrderedDict{String, Vector{UInt16}}(Vector{UInt16});
julia> d2 = DefaultOrderedDict{String, Vector{UInt16}}(Vector{UInt16});
julia> @which merge(d, d2)
merge(d::AbstractDict, others::AbstractDict...) in Base at abstractdict.jl:311
julia> @which merge(OrderedDict(d), OrderedDict(d2))
merge(d::OrderedDict, others::AbstractDict...) in OrderedCollections at /home/tlnagy/.julia/packages/OrderedCollections/PRayh/src/ordered_dict.jl:466
Because the regular dict merge function is called the order is lost when merging two DefaultOrderedDicts.
- Lenguaje dominante
- Julia
- Estrellas
- 747
- Forks
- 261
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de JuliaCollections/DataStructures.jl
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
append!(l1::MutableLinkedList, l2::MutableLinkedList) fails for empty l2Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
JuliaCollections/DataStructures.jl#997 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
-
several `mutable struct` type definitions do not require mutability, review and adjustPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Todos los issues de JuliaCollections/DataStructures.jl
Issues similares
-
lux
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
JuliaGraphs/GraphNeuralNetworks.jl#732 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
erfinv(::Float16) and erfcinv(::Float16) are wrong for abs(x) <= 0.75 (erfinv(Float16(0)) == 0.2201)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
JuliaMath/SpecialFunctions.jl#571 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100