Opened 16 years ago
Closed 12 years ago
#4328 closed Patch - Feature (Won't Fix)
support movable antennas with HD Home Run
| Reported by: | Owned by: | danielk | |
|---|---|---|---|
| Priority: | minor | Milestone: | unknown |
| Component: | MythTV - HDHomeRun | Version: | Master Head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
The attached patch adds the ability to spin around an antenna when tuning HDHR devices. It's similar to the external channel changer script for SD recorders but two extra bits of info are passed to the script - the mpeg program number and the station's callsign. The idea here is you're going to be grepping into a table of channels and azimuths so you can have your choice of how to match. Callsign makes the most sense but whatever.
Because HDHRs share their settings classes with DVB recorders this option is also exposed for them. It might be useful for someone with DVB-T but I didn't want to mess around with DVBChannel::Tune without such a device to test with.
Attachments (4)
Change History (13)
Changed 16 years ago by
| Attachment: | channelChange.patch added |
|---|
Changed 16 years ago by
| Attachment: | channelChange.15174.patch added |
|---|
comment:1 Changed 16 years ago by
| Owner: | changed from Isaac Richards to danielk |
|---|---|
| Status: | new → assigned |
| Version: | unknown → head |
comment:2 Changed 16 years ago by
| Type: | patch → enhancement |
|---|
comment:3 Changed 16 years ago by
A P.S. to this: when I wrote the patch I didn't know the multirec branch had new code to rotate antennae. At some point I plan to go back and make the HDHR work with the that stuff and submit a revised patch. Probably in the 1-2 month timeframe.
comment:4 Changed 16 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | assigned → closed |
George, I'm going to close this. Please reopen when your patch is ready.
comment:5 Changed 13 years ago by
| Resolution: | invalid |
|---|---|
| Status: | closed → new |
Lets see if I get the award for longest interval between ticket comments.
This updated patch is mostly straitforward but I want to call out the change to DTVChannel::SetChannelByString? where currently the channel change script is being used in lieu of tuning but I rework it to become part of tuning process. Well, not for S-Video and Composite inputs which continue the old way but digital streams are affected which is the point of the enhancement.
With this I'm not sure how things will shake out for firewire & I don't have such a creature to test against so I only note a possible problem.
I haven't included any UI elements but if you want 'em I'll make some.
Changed 13 years ago by
| Attachment: | hdhr.myth.v0.25pre-2256-g4d4eaa8.mythtv.patch added |
|---|
Current as of June 13
Changed 13 years ago by
| Attachment: | hdhr.myth.v0.25pre-2728-gc803867.mythtv.patch added |
|---|
VERBOSE -> LOG change broke the prior patch
comment:6 Changed 13 years ago by
Don't think this patch is high up on the list for reviews but just in case I want to call a hold or even close on it. The rec2 merge broke the logic and I haven't had a chance to look at it & who knows when I will.
comment:7 Changed 12 years ago by
| Component: | mythtv → MythTV - HDHomeRun |
|---|---|
| Type: | enhancement → Patch - Feature |
| Version: | head → Master Head |
comment:8 Changed 12 years ago by
Hmph, I was just thinking about posting a comment to this ticket to close it as it's now pretty easy to control a rotor through via script attached to the RecStarted? event.
It's better than code too, a while ago I switched my rotor input from an HDHR to an HVR-1600 and only had to touch the script's cardID test. System events has to be one of the all-time great changes!
comment:9 Changed 12 years ago by
| Resolution: | → Won't Fix |
|---|---|
| Status: | new → closed |
Thanks for the ping George
updated to apply after a recent MingGW change