I have an automation to send me a notification when Transmission finished downloading a torrent. I want to ad an action in this automation that uses the remove_torrent service and remove the torrent that was just finished. I do not know how to write the code to get the correct ID of that specific torrent. This is how my code looks when I did a very poor effort. Thanks in advanced for all your help!
Invalid config for [automation]: expected dict for dictionary value @ data[āactionā][1][ādata_templateā]. Got None. (See /config/configuration.yaml, line 108).
Iād like to make the same automation. I expected the ID of the completed torrent to be available at {{trigger.event.data.id}} based on the docs but I see that only name is provided. This is a feature request I guess.
This should work, but Iām sure could be written better.
alias: Remove torrent on successful download
trigger:
platform: event
event_type: transmission_downloaded_torrent
action:
service: transmission.remove_torrent
data:
name: Transmission
id: >
{% for torrent in state_attr('sensor.transmission_completed_torrents',
'torrent_info').items() %}{{ torrent[1].id | int }}{% endfor %}
Aside from the feature request, I think ideally we would want something like this:
id: "{{ states.sensor.transmission_completed_torrents.attributes.torrent_info[trigger.event.data.name].id | int }}"
The error message I can see in my log looks like this.
Logger: homeassistant.components.automation.completed_torrent
Source: core.py:1405
Integration: Automation ([documentation](https://www.home-assistant.io/integrations/automation), [issues](https://github.com/home-assistant/home-assistant/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+automation%22))
First occurred: 9:05:53 (2 occurrences)
Last logged: 9:10:50
While executing automation automation.completed_torrent
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/automation/__init__.py", line 433, in async_trigger await self.action_script.async_run(
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 1010, in async_run await asyncio.shield(run.async_run())
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 245, in async_run await self._async_step(log_exceptions=False)
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 253, in _async_step await getattr(
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 460, in _async_call_service_step await service_task
File "/usr/src/homeassistant/homeassistant/core.py", line 1405, in async_call processed_data = handler.schema(service_data)
File "/usr/local/lib/python3.8/site-packages/voluptuous/schema_builder.py", line 272, in __call__ return self._compiled([], data)
File "/usr/local/lib/python3.8/site-packages/voluptuous/schema_builder.py", line 594, in validate_dict return base_validate(path, iteritems(data), out)
File "/usr/local/lib/python3.8/site-packages/voluptuous/schema_builder.py", line 432, in validate_mapping raise er.MultipleInvalid(errors)
voluptuous.error.MultipleInvalid: expected int for dictionary value @ data['id']
I do get the notification to my phone correctly but the torrent is not removed. I can also see that my sensor.transmission_total_torrents does contain the correct downloaded torrents but still they are not removed.
I can see indentation and spaces in your example. It needs to be copy/pasted exactly as I had to ensure an integer is used as the value
Like I say this is very poorly written, need someone who knows Python/this template syntax to improve it, and even with better syntax itās still not ideal, the second example is the one to strive for.
I should also mention in case you donāt know, that if you donāt want to seed torrents you can just set the seeding ratio to 0.
Iāve looked a bit closer at this and figured out the prefered solution which is much better. It seems we must wait a couple of seconds for the sensor to be updated when the event fires.
Let us know if access to any other attributes would be useful! Anything you can think of is possible but we want to keep it limited to generally useful attributes (e.g. seed ratio, start date, end date, etc.).
Here is the specification:
Here is a cool frontend for Transmission if you use HACS or can install custom component:
To automate file processing, I would love to have access from the event trigger data to the downloadDir property of the torrent. Hard-coding the default download directory in a template is just to fragile.
After reading that Transmission integration was updated I tried to change your code as seen below but it did not work for me. Have you been able to update your working code with their new updates?
@weckma the changes linked above are yet to be merged, and then they need to be included in a Home Assistant release. Iāll update this thread when the changes are available to use, then your example should work.
By the way I believe due to the excellent changes @JPHutchins is making, the delay step wonāt be necessary.
Iām a bit late to the party but I canāt get this to work. I have the notification part nailed down but cannot work out how to delete the file - i get this error: