Opened 18 years ago
Closed 14 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)
by , 18 years ago
| Attachment: | channelChange.patch added |
|---|
by , 18 years ago
| Attachment: | channelChange.15174.patch added |
|---|
comment:1 by , 18 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
| Version: | unknown → head |
comment:2 by , 18 years ago
| Type: | patch → enhancement |
|---|
comment:3 by , 18 years ago
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 by , 17 years ago
| Resolution: | → invalid |
|---|---|
| Status: | assigned → closed |
George, I'm going to close this. Please reopen when your patch is ready.
comment:5 by , 14 years ago
| 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.
by , 14 years ago
| Attachment: | hdhr.myth.v0.25pre-2256-g4d4eaa8.mythtv.patch added |
|---|
Current as of June 13
by , 14 years ago
| Attachment: | hdhr.myth.v0.25pre-2728-gc803867.mythtv.patch added |
|---|
VERBOSE -> LOG change broke the prior patch
comment:6 by , 14 years ago
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 by , 14 years ago
| Component: | mythtv → MythTV - HDHomeRun |
|---|---|
| Type: | enhancement → Patch - Feature |
| Version: | head → Master Head |
comment:8 by , 14 years ago
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!
updated to apply after a recent MingGW change