Em casos assim, você poderia ter feito o teste de mesa, que teria percebido facilmente os problemas do seu código: você não troca o sinal para alternar entre subtração e soma, e o loop mais interno é completamente desnecessário.
Dá pra fazer assim:
def soma():
fat = -1
total = 0
for i in range(1, 11):
fat = -fat * i
total += (11 - i) / fat
return round(total, 2)
E só.
Outras respostas sugeriram ter uma função só pra calcular o fatorial, que também funciona, mas neste caso eu acho desnecessário. Por exemplo, se no meio do cálculo você já fez o fatorial de 3, quer dizer que já multiplicou 1, 2 e 3. Depois, na próxima iteração terá que calcular o fatorial de 4, e multiplicará novamente 1, 2 e 3, para multiplicar por 4.
Ou seja, serão feitas várias multiplicações redundantes e desnecessárias (isso acontece mesmo se usar a função pronta do módulo math
, pois apesar dela usar um algoritmo mais eficiente, ainda sim acaba tendo esse retrabalho desnecessário). Uma das respostas ainda sugere usar recursão, que torna tudo mais ineficiente ainda (sei que fatorial é muito usado como exemplo para ensinar recursão, mas muitos esquecem de mencionar que não é a forma mais eficiente de se fazer este cálculo).
No caso do seu cálculo, basta ir multiplicando o último fatorial encontrado pelo próximo número, e invertendo o sinal para que ele alterne entre a soma e a subtração.
Como curiosidade, segue um comparativo entre as soluções: https://ideone.com/nyUKyd - repare como a solução recursiva é mais lenta (claro que para cálculos pequenos executados poucas vezes não faz diferença, mas é importante se atentar a esses detalhes e não sair usando recursão onde não precisa).