Boas práticas

Seria uma boa prática criar uma variável para o event.target.parentNod? No caso no código ele é repetido duas vezes.

var tabela = document.querySelector("table");

tabela.addEventListener("dblclick", function(event){ 
    if (event.target.tagName == 'TD') {
            event.target.parentNode.classList.add("fadeOut");
            setTimeout(function() {
              event.target.parentNode.remove();
        }, 500);
    }
});

Acima ele se repete duas vezes na função, então eu criei uma variável chamada paciente, por que basicamente o event.target.paretNode pega um paciente da tabela e remove, e o código ficou assim:

var tabela = document.querySelector("table");

tabela.addEventListener("dblclick", function(event){ 
    var paciente = event.target.parentNode;
    if (event.target.tagName == 'TD') {
            paciente.classList.add("fadeOut");
            setTimeout(function() {
              paciente.remove();
        }, 500);
    }
});

Enfim, seria uma boa prática? ou é meio que indiferente.

Valeu, abs