Este código não faz muito sentido, está todo "atropelado". Já começa com um problema que não causa erro. Pra que transformar um string em string?
Pra que separar as vogais com vírgula se você só precisa colocar as vogais para fazer a verificação se a letra se encontra lá?
Se o problema pede para verificar se a letra é vogal porque o teste é feito em cima da palavra toda? E porque converte toda a palavra para maiúscula e depois vê se ela está entre as vogais. Entre as vogais só não, entre as vogais, a vírgula ou espaço em branco. E nunca vai estar por duas razões: você converte para maiúsculo e depois olha onde só tem minúsculas, não tem como dar certo. Mas é pior porque está pedindo para ver se a palavra inteira está entre as vogais, clara que uma palavra não está a não ser que a palavra seja só uma vogal, ou uma sequência que está lá.
Então precisa varrer a palavra pegando letra por letra e testar se a letra, e não a palavra, está entre as vogais, e só as vogais, mais nada. Se estiver, aí você imprime com upper()
, ou caso contrário deve imprimir também sem mudar a letra, não pode só jogar a variável lá sem dizer que o que fazer com ela.
Do jeito que está vai colocar uma letra em cada linha, precisaria formatar para não pular linha, mas deixo isso como exercício final até porque não deixa claro se precisa colocar tudo na mesma linha.
Tem outras formas mais simples de fazer isso, mas para começar acho que está bom assim:
vogais = 'aeiou'
palavra = input('Digite uma palavra: ')
for letra in palavra:
if letra in vogais:
print(letra.upper())
else:
print(letra)
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
y
nas vogais.