lilott8 / lilott8/BioScript

VolumeTracker should deal with Phi nodes appropriately

Abierto
#79 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

bug good first issue
Lenguaje dominante
Python
Estrellas
53
Forks
16
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

The volume tracker does not appropriately handle phi nodes.

The issue arises when a phi node exists in a loop header, where there is a use prior to a def, and has the following structure:

before_loop:
    a0 = ...
loop_header:
    a1 = phi(a0, a1)
loop_body:
    ... = use a1

A test case has been added to the volume tracking testing harness using tests/assays/probabilistic_pcr.bs as the model.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza con el arnés de pruebas de seguimiento de volumen y el caso de prueba existente en tests/assays/probabilistic_pcr.bs. Reproduce el escenario de phi en la cabecera del bucle con un uso antes de la definición y, después, sigue cómo el rastreador de volumen gestiona el nodo phi. La tarea estará terminada cuando la prueba añadida pase y esta estructura de bucle se gestione adecuadamente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
compilers
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
48/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.