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 (mesmo se usar a função pronta do módulo math
, pois internamente ela vai fazer tudo isso de novo). 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).