stride3d/stride

[Animation] Messy curve/animation/evaluator API

Open

#449 aberto em 18 de mai. de 2019

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C# (886 forks)batch import
area-Graphicsenhancementgood first issue

Métricas do repositório

Stars
 (6.001 stars)
Métricas de merge de PR
 (Mesclagem média 10d 11h) (22 fundiu PRs em 30d)

Description

Release Type: Github

Version: Latest

Platform(s): Any

Describe the bug The animation/curve API as it is is not very maintainable and quite restrictive, there's no reason now that generics supports unmanaged and interface type constraints to have two implementation of a class differing in only the generic type they chose, ex : AnimationCurveEvaluatorOptimizedVector3Group / AnimationCurveEvaluatorOptimizedVector4Group. We can replace 2/3 of the files within /Animations by implementing and adding an IInterpolation interface on our math types and including it as a constraint on our generics requiring that kind of features.

Guia do colaborador

Direção de pesquisa
Examine as classes AnimationCurveEvaluatorOptimized* atuais para identificar padrões comuns e, em seguida, projete uma classe genérica com uma restrição de interface IInterpolation para reduzir a duplicação de código.
Pilha de tecnologia
csharp
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#generics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30