IntersectMBO/plutus

The evaluate-builtins optimization isn't conservative

Open

#6.167 aberto em 3 de jun. de 2024

Ver no GitHub
 (1 comment) (1 reaction) (0 assignees)Haskell (511 forks)batch import
Good first issueInternalPlutus IRbugoptimizationstatus: triagedtech debt

Métricas do repositório

Stars
 (1.637 stars)
Métricas de merge de PR
 (Mesclagem média 15d 11h) (13 fundiu PRs em 30d)

Description

PlutusIR.Transform.EvaluateBuiltins turns ifThenElse True x y into x. That is not conservative: if y throws then the former term will throw and the latter one won't. So this is a bug that needs to be fixed. It wasn't discussed in the original PR.

More importantly, this looks like something that the proper property testing should catch. Look like a perfect opportunity for the PIR generators to show their power. So it'd be best to first write a failing property, then fix the bug.

Guia do colaborador

Direção de pesquisa
Escreva um teste baseado em propriedades que falhe para a otimização evaluate builtins e corrija o bug tornando a otimização conservadora.
Pilha de tecnologia
None
Domain
blockchain
Tipo Issue
Bug
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-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
HaskellPlutusProperty testing
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65