MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 38/100
- Tipo de issue
- Refatoração
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- cpp
- Domínio
- computer-graphics, performance
Direção de pesquisa
Comece localizando a implementação existente de Clamp() e todos os seus pontos de chamada, depois compare-os com a referência vinculada de C++17 std::clamp. Considera-se concluído quando for determinado se a substituição preserva o desempenho e o comportamento para a ordenação dos argumentos NaN e para +0.0 em comparação com -0.0, com todos os usos afetados identificados.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
As we are now C++17 or greater we should probably investigate replacing calls to Clamp() with std::clamp.
Care may need with both performance and behaviour of non-normals etc to ensure the behaviour remains unchanged - for example ordering of arguments when one of them is a Nan, handling of +0.0 vs -0.0 etc
https://en.cppreference.com/w/cpp/algorithm/clamp.html
At first glance our implementation appears to be directly substitutable.
- Linguagem predominante
- C++
- Estrelas
- 2.1k
- Forks
- 508
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de AcademySoftwareFoundation/OpenColorIO
-
Inconsistent run-time library usage for locale handling breaks GPU code generation on some systems.Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 86/100
AcademySoftwareFoundation/OpenColorIO#2351 · 2 comentários ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
-
Build failure on GCC/MinGW (MXE): std::ifstream constructor mismatch with Platform::filenameToUTF()Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
-
ReadTheDocs - OS DeprecationAbertaDocumentation
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 72/100
AcademySoftwareFoundation/OpenColorIO#2280 · 1 comentário ·
Todas as issues de AcademySoftwareFoundation/OpenColorIO
Issues semelhantes
-
parse_selu lowering loses catastrophic cancellation → silent 0.0 for moderate positive inputsAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
ROCm/AMDMIGraphX#5490 ·
Mantenedores costumam responder em até 2 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
arvidn/libtorrent#9126 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Sunrisepeak/mcpp-language-server#80 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
xiaoyifang/goldendict-ng#3148 ·
Mantenedores costumam responder em até 1 dia