| Wednesday, August 3rd, 2011, 00:09 UTC | ||
| [00:09:57] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 255 seconds) | |
| [00:33:14] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [00:54:44] | andreax (andreax!~andreaz@p57B92641.dip.t-dialin.net) has quit (Quit: Leaving.) | |
| [01:02:42] | andreax (andreax!~andreaz@p57B92641.dip.t-dialin.net) has joined #mythtv | |
| [01:03:17] | gigem_ (gigem_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [01:03:45] | gigem_ (gigem_!~david@host103.16.intrusion.com) has joined #mythtv | |
| [01:03:45] | gigem_ (gigem_!~david@host103.16.intrusion.com) has quit (Changing host) | |
| [01:03:45] | gigem_ (gigem_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [01:06:30] | andreax (andreax!~andreaz@p57B92641.dip.t-dialin.net) has quit (Read error: Connection reset by peer) | |
| [01:09:20] | paul-h (paul-h!~paulh@mythtv/developer/paul-h) has quit (Ping timeout: 252 seconds) | |
| [01:10:35] | paul-h (paul-h!~paulh@5adce259.bb.sky.com) has joined #mythtv | |
| [01:10:35] | paul-h (paul-h!~paulh@5adce259.bb.sky.com) has quit (Changing host) | |
| [01:10:36] | paul-h (paul-h!~paulh@mythtv/developer/paul-h) has joined #mythtv | |
| [01:21:11] | paul-h (paul-h!~paulh@mythtv/developer/paul-h) has quit (Read error: Operation timed out) | |
| [01:23:47] | paul-h (paul-h!~paulh@5adce259.bb.sky.com) has joined #mythtv | |
| [01:23:47] | paul-h (paul-h!~paulh@5adce259.bb.sky.com) has quit (Changing host) | |
| [01:23:47] | paul-h (paul-h!~paulh@mythtv/developer/paul-h) has joined #mythtv | |
| [01:35:34] | paul-h (paul-h!~paulh@mythtv/developer/paul-h) has quit (Ping timeout: 264 seconds) | |
| [01:36:13] | Meliorator (Meliorator!m@dunnington.eu) has quit (Read error: Connection reset by peer) | |
| [01:38:26] | paul-h (paul-h!~paulh@5adce259.bb.sky.com) has joined #mythtv | |
| [01:38:26] | paul-h (paul-h!~paulh@5adce259.bb.sky.com) has quit (Changing host) | |
| [01:38:26] | paul-h (paul-h!~paulh@mythtv/developer/paul-h) has joined #mythtv | |
| [01:40:18] | Meliorator (Meliorator!m@dunnington.eu) has joined #mythtv | |
| [01:41:26] | Beirdo: | I think I got the magic fix on the mpeg2fix that eluded me last night. Need to clean it up and test a bit more :) |
| [01:56:24] | danielk22: | Beirdo: w00t! :) |
| [02:06:38] | NightMonkey (NightMonkey!~NightMonk@pdpc/supporter/professional/nightmonkey) has joined #mythtv | |
| [02:07:18] | danielk22: | FYI I think I've got the mthread stuff all worked out. I'm just going to install it on my production systems and run it for a few days before I push it. |
| [02:08:11] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [02:22:24] | danielk22: | sphery: i was thinking about an angle on the mysql deadlocks.. what if qt is not reissuing prepare statements after a reconnect.. then when it tries to execute a prepared statement that is no longer there things go boom. |
| [02:23:35] | danielk22: | sphery: this of course presumes there have been some changes in the sql driver since we first ported to Qt4 since it appears our users didn't start experiencing these problems until the last year or so. |
| [02:28:12] | cesman (cesman!~cecil@pdpc/supporter/professional/cesman) has quit (Remote host closed the connection) | |
| [02:28:40] | Beirdo: | danielk22: sounds good :) |
| [02:31:04] | cesman (cesman!~cecil@pool-108-38-214-203.lsanca.fios.verizon.net) has joined #mythtv | |
| [02:31:04] | cesman (cesman!~cecil@pdpc/supporter/professional/cesman) has joined #mythtv | |
| [02:31:04] | cesman (cesman!~cecil@pool-108-38-214-203.lsanca.fios.verizon.net) has quit (Changing host) | |
| [02:36:24] | sphery: | danielk22: it definitely doesn't re-issue a prepare after a reconnect. There have been several changes to prepared statement handling... In Qt 4.4.x and below, prepared statements were only used if the developer asked for them through QSqlQuery::prepare(), in 4.5.0–4.5.2, they got used for every query, whether we asked or not, and Qt 4.5.3 fixed that bug, so in theory it and all later ones only use prepared statements when requested. |
| [02:37:15] | sphery: | I was starting to think it's the same thing--and that the MYSQL_OPT_RECONNECT has just made it so that we get the same result with TCP/IP connections that we were getting, before, only with local socket connections. |
| [02:37:48] | sphery: | the fact that several users are posting logs with SQL that contains our bind variables seems to indicate that we're issuing "unprepared" SQL statements |
| [02:39:32] | danielk22: | sphery: we can probably test this theory by forcing a disconnect after every prepare statement and seeing what happens... |
| [02:40:34] | sphery: | unfortunately, refactoring our SQL usage (which is spread throughout all of MythTV) to work in the face of lost/re-connected statements and able to re-prepare/re-bind statements will be quite a challenge |
| [02:41:22] | sphery: | might just be time to make that mythdataserver (and let it do all the data access and package results into data transfer objects to be sent back using some serialization format) :) |
| [02:41:23] | danielk22: | sphery: we'd just do it in the wrapper.. we can cache the last prepare and resend if we got a disconnect. |
| [02:41:37] | sphery: | we'd have to cache the prepare and all the binds, though, right? |
| [02:42:51] | danielk22: | sphery: only the last bind.. that's no biggie. it's just a pointer copy and a reference count increment. |
| [02:42:55] | sphery: | oh, and FWIW, Qt 3.x never used prepared statements with MySQL--even if you did QSqlQuery::prepare() (the drivers faked it with string substitutions) |
| [02:43:19] | sphery: | well, last bind for each bind value... so as many binds as there are bind variables |
| [02:43:24] | danielk22: | sphery: we didn't really see this in the first 2 years with Qt4 though.. |
| [02:43:44] | sphery: | yeah, I don't know why we're seeing it so often, now--where "we" does not include me :) |
| [02:44:05] | danielk22: | sphery: right, that's what i mean by the last bind.. it's a single container internally. |
| [02:45:03] | danielk22: | QMap<QString, QVariant> |
| [02:46:30] | danielk22: | I think QSqlQuery.boundValues() would still be valid so we probably don't even need the assignment.. |
| [02:47:41] | danielk22: | heh, and we already cache the last prepared query in MSqlQuery::m_last_prepared_query :) |
| [02:47:57] | sphery: | cool |
| [02:48:02] | sphery: | so the challenge is detecting it? |
| [02:48:37] | sphery: | especially with our 4 or 5 places that are trying to cope with database going missing |
| [02:50:08] | danielk22: | yeah.. well when we do the reconnect it's easy enough for us to know it happened.. with the automatic reconnect i dunno.. i know we could do it with a temp table, but that wouldn't really be efficient at all. |
| [02:50:29] | sphery: | yeah, I think we should disable auto reconnect if we have code that properly handles it |
| [02:51:14] | sphery: | that's exactly why mysql changed their defaults--to allow apps that are written to handle it to detect the loss of a connection and clean up |
| [02:52:08] | danielk22: | ah.. ok. It's strange that they didn't think of prepare/disconnect/reconnect/exec seems like something that should be handled in Qt. |
| [02:53:43] | danielk22: | Anyway getting late here.. Maybe I can whip up something tomorrow.. |
| [02:54:38] | sphery: | Yeah, that would definitely be nice. I think the MYSQL_OPT_RECONNECT was added only because it was a tiny change and most Qt apps had been written with the presumption that they wouldn't know when a statement is reconnected. Though I'm guessing that now that Qt is actually using prepared statements for prepared statements with MySQL, many apps are finding that the auto-reconnect doesn't work like it did in Qt3. :) |
| [02:57:49] | sphery: | (So, Qt gave the users what they asked for, but it's not what they wanted.) |
| [03:01:29] | Beirdo: | watching my TS transcoded file. so far so good |
| [03:18:40] | taylorr: | Beirdo: what was the fix for source material with repeated frames? |
| [03:23:23] | Beirdo: | turns out there was code in there to deal with it already, but... I had messed up the reordering code |
| [03:23:59] | Beirdo: | basically, it sets the PTS change to be 3 frames instead of 2 |
| [03:24:54] | Beirdo: | and in the reordering, I missed preincrementing the index within the list to skip the first frame (which gets moved to after the B frames |
| [03:31:19] | Beirdo: | don't think I have it 100%, but it's sure a lot better :) |
| [03:34:26] | Beirdo: | have three hacks to try backing out before squashing this into a patch for a review |
| [04:02:37] | xris: | odd no one else replied to my review board email |
| [04:02:53] | xris: | reviewboard. stupid osx autocomplete. |
| [04:05:32] | Beirdo: | heh, well, we'll see if anyone does tonight, I guess |
| [04:44:44] | Beirdo: | danielk22: BTW, you have git setup to use your cuymedia.net rather than mythtv.org address |
| [04:45:25] | Beirdo: | just in case ya care :) We are trying to standardize it to use the mythtv.org ones, but at least you have a valid one there :) |
| [05:12:40] | Beirdo: | danielk22: stuartm: http://cgit.ipv6.beirdo.ca/git/mythtv/commit/?h=q3ptrlist |
| [05:13:35] | Beirdo: | since we don't have a good home for it at this time without pushing it to a public branch... that's what I have giving me working output from my OTA (MPEG2-TS) and analog (MPEG2-PS) recordings |
| [05:14:11] | Beirdo: | oh, maybe I should give you the NON-IPv6 URL :) |
| [05:14:24] | Beirdo: | http://www.beirdo.ca/git/mythtv/commit/?h=q3ptrlist |
| [05:54:21] | gandalfcome (gandalfcome!~gandalfco@mithrandir.anu.edu.au) has joined #mythtv | |
| [06:33:21] | dekarl (dekarl!~dekarl@dslb-084-058-164-202.pools.arcor-ip.net) has joined #mythtv | |
| [07:09:46] | NightMonkey (NightMonkey!~NightMonk@pdpc/supporter/professional/nightmonkey) has quit (Ping timeout: 250 seconds) | |
| [07:35:59] | kormoc is now known as kormoc_afk | |
| [08:22:37] | andreax (andreax!~andreaz@p57B92641.dip.t-dialin.net) has joined #mythtv | |
| [08:24:42] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has quit (Quit: Leaving.) | |
| [10:05:01] | mike|3 (mike|3!~mike@c-24-21-63-118.hsd1.or.comcast.net) has quit (Remote host closed the connection) | |
| [10:05:55] | mike|2 (mike|2!~mike@c-24-21-63-118.hsd1.or.comcast.net) has joined #mythtv | |
| [10:27:18] | Gibby (Gibby!~Gibby@204.118.10.244) has quit (Ping timeout: 276 seconds) | |
| [10:28:53] | Gibby (Gibby!~Gibby@204.118.10.244) has joined #mythtv | |
| [11:15:50] | andreax (andreax!~andreaz@p57B92641.dip.t-dialin.net) has quit (Read error: Connection reset by peer) | |
| [12:04:59] | gigem_ (gigem_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [12:05:27] | gigem_ (gigem_!~david@host103.16.intrusion.com) has joined #mythtv | |
| [12:05:27] | gigem_ (gigem_!~david@host103.16.intrusion.com) has quit (Changing host) | |
| [12:05:27] | gigem_ (gigem_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [12:19:14] | JamesHarrison (JamesHarrison!~jharrison@cpc3-addl4-2-0-cust97.6-3.cable.virginmedia.com) has quit (Ping timeout: 252 seconds) | |
| [12:29:45] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has joined #mythtv | |
| [12:32:33] | JamesHarrison (JamesHarrison!~jharrison@cpc3-addl4-2-0-cust97.6-3.cable.virginmedia.com) has joined #mythtv | |
| [12:47:07] | JamesHarrison (JamesHarrison!~jharrison@cpc3-addl4-2-0-cust97.6-3.cable.virginmedia.com) has quit (Ping timeout: 240 seconds) | |
| [13:16:06] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has joined #mythtv | |
| [13:21:42] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [13:27:05] | gandalfcome (gandalfcome!~gandalfco@mithrandir.anu.edu.au) has quit (Read error: Connection reset by peer) | |
| [13:27:30] | yoyolala (yoyolala!~yoyolala@unaffiliated/yoyolala) has quit (Quit: leaving) | |
| [13:34:02] | PointyPumper (PointyPumper!~pintlezz@190.244.73.13) has quit (Ping timeout: 252 seconds) | |
| [13:41:04] | j-rod|afk is now known as j-rod | |
| [13:45:32] | reynaldo (reynaldo!~rverdejo@pc-161-56-101-190.cm.vtr.net) has joined #mythtv | |
| [14:10:36] | danielk22: | Beirdo: Re: Q3PtrList. We use NULL for null pointers instead of 0 these days, for all the reasons C++1x introduces nullptr to the language.. |
| [14:17:34] | danielk22: | Beirdo: "mp2x33(af->at(1)->pkt.pts," this doesn't look right.. you just checked that at(0) was valid with (af->count() > 1) but then pulled at(1)... |
| [14:21:59] | danielk22: | now i remember one of the reasons this didn't get converted before, it uses at(x) not just to get an item but also relies on it to update an implicit index. :P |
| [14:26:25] | danielk22: | Beirdo: ok thats it on the things that really need to be looked at, the NULL ptr thing was "displayFrame = 0;" at about line 199. |
| [14:28:49] | danielk22: | Style wise I'd use "size()" instead of "count()" since the first is more STL like and I sometimes temporarily switch Qt containers to STL ones when I suspect the Qt reference counting is getting bulloxed. |
| [14:30:38] | danielk22: | Also I'd skip the typedefs for FrameListIterator and FrameMapIterator and just use FrameList::iterator and FrameMap::iterator .. When I see a typedef for an iterator I always assume it's non-standard and I need to look at how they are defined. If I see a standard construction I assume standard behavior. |
| [14:32:34] | danielk22: | The suspect af(1) is in MPEG2fixup::FindStart() |
| [14:36:29] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [14:36:43] | NightMonkey (NightMonkey!~NightMonk@pdpc/supporter/professional/nightmonkey) has joined #mythtv | |
| [14:39:46] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 240 seconds) | |
| [14:41:31] | stuartm: | danielk22: right, the container maintained an internal iterator which always pointed at the last used item (accessed via current() ), it was a headache to untangle that and I didn't even finish my attempt, hence why Beirdo was having a go |
| [15:02:12] | andreax (andreax!~andreaz@p57B92641.dip.t-dialin.net) has joined #mythtv | |
| [15:02:37] | yoyolala (yoyolala!~yoyolala@unaffiliated/yoyolala) has joined #mythtv | |
| [15:17:23] | zombor_ is now known as zombor | |
| [15:19:22] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [15:20:22] | danielk22: | iamlindoro: For the channel scanner I need to update the web page as the scanner progresses, what is the proper way to do that? Some javascript that requests the current state? And if so are we using some standard encapsulation? MythServices? |
| [15:21:46] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 240 seconds) | |
| [15:22:33] | iamlindoro: | danielk22: Great question-- I have been picking up the javascript stuff as I've needed to learn it, and haven't yet run into anything where I needed to run it asynchronously/updating. For the source of data, though, we've been using the new services API, which returns the data as XML or JSON, and using JSON... JQuery has a JSON get method, so I'll bet they also have a means of doing periodic updates too |
| [15:23:32] | danielk22: | We're using jQuery in our setup pages? |
| [15:23:36] | iamlindoro: | yeah |
| [15:23:50] | zombor_ is now known as zombor | |
| [15:24:19] | danielk22: | k, I'll need to figure out how to hook into the new services API.. |
| [15:24:45] | sraue_ (sraue_!~stephan@6-46-239-77-pool.cable.fcom.ch) has joined #mythtv | |
| [15:25:03] | sraue (sraue!~stephan@xbmc/staff/sraue) has quit (Read error: Connection reset by peer) | |
| [15:25:23] | iamlindoro: | It's pretty easy once you've got the service set up, but obviously that's the trick... the scanner is definitely an order of magnitude more complicated than the stuff we've done so far :) |
| [15:25:39] | danielk22: | What's a good example service? |
| [15:26:21] | iamlindoro: | I suspect you'd want to add a whole new service rather than add methods to any of the existing ones, so let me see if I can finda good commit |
| [15:28:05] | danielk22: | thx |
| [15:29:51] | iamlindoro: | danielk22: This is a service I wrote, so doubtless it's not the best example of anything ;) But at least it shows you all the pieces that are needed to add a service, and it follows the coding style of the initial services very, very closely. |
| [15:29:52] | iamlindoro: | https://github.com/MythTV/mythtv/commit/5e712 . . . e8e7f04ad251 |
| [15:40:06] | iamlindoro: | danielk22: FWIW there's a thread going on the users list right now that's more than a little discourteous about the scanner-- but points out some long standing bugs (attributes get reset on a rescan, channels are missed when dvbscan finds them, etc)-- I responded and pointed out the tickets, but you may have more to add if interested |
| [15:40:16] | iamlindoro: | (Re: [mythtv-users] is (clearqam) channel scanning idempotent (NO!)?) |
| [15:41:37] | stuarta: | it's not the best thing on the planet |
| [15:41:56] | stuarta: | works better in some countries than others |
| [15:42:17] | danielk22: | *shrug* I think it's better to spend my time fixing the scanner than discussing it, I know there are major issues. |
| [15:42:37] | ** stuarta me too's ** | |
| [15:43:24] | iamlindoro: | danielk22: No worries, I just am never quite sure if I've made an accurate argument-- I've tried to fix a few little things when possible, but the existing tickets are things that I've looked and and have eluded me in the past |
| [15:43:45] | iamlindoro: | It was more or less that I'm sure there are compelling and correct explanations that I just plain don't know about |
| [15:45:43] | danielk22: | It's just buggy and incomplete. I had to apply it pre-maturely because the old UI couldn't complete a scan without crashing. Then I had no time to work on it. |
| [16:06:05] | andreax1 (andreax1!~andreaz@p57B94FE8.dip.t-dialin.net) has joined #mythtv | |
| [16:06:40] | andreax (andreax!~andreaz@p57B92641.dip.t-dialin.net) has quit (Ping timeout: 250 seconds) | |
| [16:30:09] | Beirdo: | danielk22: displayFrame is no longer a pointer, it's an index |
| [16:30:51] | Beirdo: | I'll take a look at the at(1), you may be correct there |
| [16:31:21] | gregL (gregL!~greg@cpe-74-76-125-87.nycap.res.rr.com) has quit (Ping timeout: 260 seconds) | |
| [16:33:15] | Beirdo: | the at(1) in question in intentional to follow the functionality of the old code |
| [16:33:43] | Beirdo: | af->next()-> became af->at(1)-> |
| [16:33:56] | Beirdo: | as af was always at the beginning of the list |
| [16:35:25] | Beirdo: | it does seem odd though. |
| [16:35:53] | Beirdo: | I would have thought af->first() would have been what we'd want, but that doesn't match the old code |
| [16:38:15] | Beirdo: | I'll look over the rest of the comments a bit later, I should probably head for work :) Thanks for having a look. |
| [16:41:32] | danielk22: | Wouldn't at(0) be the top of the list? |
| [16:56:17] | danielk22: | That is either it should be at(0) there or the check above should be "if (delta < 0 && af->count() > 2)" since the count needs to be at least 2 for at(1) to be referencing uninitialized data... |
| [17:04:51] | gigem_ (gigem_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [17:05:19] | gigem_ (gigem_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [17:24:11] | gregL (gregL!~greg@cpe-74-76-115-239.nycap.res.rr.com) has joined #mythtv | |
| [17:24:38] | Mousey (Mousey!~wtfisme@ross154.net) has joined #mythtv | |
| [17:35:03] | kormoc_afk is now known as kormoc | |
| [17:46:23] | zCougar (zCougar!~cougar@2001:67c:32c:600:250:56ff:fe81:5f) has quit (Remote host closed the connection) | |
| [17:48:49] | Beirdo: | at(1) is the second item, which is the same as you'd get with a q3ptrlist at first()->next() |
| [17:49:07] | Beirdo: | now, why he wanted to test the second item... not sure |
| [17:49:44] | Beirdo: | but yeah, some tweaking there will idiot-protect it one way or the other |
| [17:52:52] | danielk22: | ok, got it. I'm wrong, count()>1 will ensure that at(1) is valid. |
| [17:53:11] | danielk22: | AFAICT The code is all correct. |
| [17:53:41] | Beirdo: | cool. I like your idea with the Iterators though |
| [17:54:03] | Beirdo: | it was out in full before, so I shortened them, but yeah ::Iterator works just fine :) |
| [17:54:56] | Beirdo: | and the FrameMap especially was quite the mouthful |
| [18:05:52] | JamesHarrison (JamesHarrison!~jharrison@cpc3-addl4-2-0-cust97.6-3.cable.virginmedia.com) has joined #mythtv | |
| [18:34:17] | danielk22: | iamlindoro: are all those datacontracts hand written? |
| [18:40:27] | Meliorator (Meliorator!m@dunnington.eu) has quit (Ping timeout: 276 seconds) | |
| [18:47:11] | wagnerrp: | yes |
| [18:48:15] | iamlindoro: | danielk22: ^^ What he said |
| [18:49:33] | gregL (gregL!~greg@cpe-74-76-115-239.nycap.res.rr.com) has quit (Ping timeout: 276 seconds) | |
| [18:49:46] | Beirdo: | heh. OK, getting rid of the Iterator typedefs, and making for, if, while follow our coding standard (which I sometimes forget, I'll try to do better) |
| [18:51:41] | iamlindoro: | danielk22: If the question was in reference to indentation/style/formatting, I followed the style of the rest of the lib for consistency |
| [18:56:34] | danielk22: | iamlindoro: yeah, it definitely doesn't follow mythtv coding standards.. but i think it's best to fix that all in one formatting commit than doing it ad-hock. |
| [18:58:10] | danielk22: | iamlindoro: I'm more concerned about the code duplication .. i can't count the number of bugs i've spent hours tracing down that turned out to be due to one of multiple duplicates of something being updated. |
| [18:58:37] | Beirdo: | yeah, fix it once, you gotta go fix it umpteen times. Not fun |
| [19:02:41] | danielk22: | iamlindoro: ok so I created a service following the channel model, how do I invoke it? |
| [19:03:01] | iamlindoro: | danielk22: I'm not sure where I duplicated code? I think the channel service mostly just calls the channel util stuff? |
| [19:03:23] | iamlindoro: | danielk22: BackendIP:6544/ServiceName/MethodName?Arg1=x&Arg2=y&Arg3=z |
| [19:03:46] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 264 seconds) | |
| [19:04:56] | iamlindoro: | Ah, you mean in the other services, the duplicated stuff |
| [19:05:23] | iamlindoro: | yeah, most of the stuff I've added uses existing functions when applicable, I think-- but there's some other stuff that's one-off |
| [19:09:11] | iamlindoro: | Note that I haven't added most of the services/methods-- Channel and Video are mine, and a couple additions to the others |
| [19:12:56] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [19:13:48] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 276 seconds) | |
| [19:16:03] | danielk22: | iamlindoro: The more I look at this code the more I feel we need to remove it all :| It's not just not following coding style. The API doesn't seem to be well thought out and it uses the bad parts of C++ like C++ exceptions. depressing |
| [19:16:51] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [19:17:19] | gregL (gregL!~greg@cpe-74-76-125-87.nycap.res.rr.com) has joined #mythtv | |
| [19:20:17] | Meliorator (Meliorator!m@dunnington.eu) has joined #mythtv | |
| [19:21:00] | danielk22: | It seems like such a good idea and a lot of energy has been expended... maybe it can be salvaged and i'm just not seeing it. |
| [19:21:41] | danielk22: | I need to stop looking at it for a while and then come back to it when I'm in a really good mood. |
| [19:21:55] | Beirdo: | that can help, yeah. |
| [19:22:36] | iamlindoro: | I definitely don't want to add comment, I just don't know enough to render an opinion, other than to say that from a "write a service and use it in a web app" perspective, I've found it really easy to use... I haven't looked into how it works behind the scenes |
| [19:23:04] | Beirdo: | so for the q3ptrlist stuff, I'll leave it for stuartm (and other interested parties) to peruse for a couple days, and then if there's no major objections, or whatever, I'll put it into master |
| [19:23:36] | Beirdo: | thank you, danielk22, for taking a read through it, I know it's not easy. |
| [19:23:39] | iamlindoro: | Like you say, a ton of work went into it, and since I am ignorant about the pros and cons of how it works-- I do know that I like the idea of it and that for purposes of adding a service that can be used in lots of different ways, it was pretty neat to me |
| [19:24:26] | danielk22: | iamlindoro: Yeah, that's the stuff I like and why I really liked the concept before I looked at the code. |
| [19:25:18] | danielk22: | It's sort of like research code.. i'm getting grad school flashbacks. :P |
| [19:25:46] | Beirdo: | the good kind, or the bad kind? :) |
| [19:26:14] | wagnerrp: | clearly your research code is significantly different than the research code im working with |
| [19:27:01] | wagnerrp: | im guessing youve never had to deal with old NASA F77, written like 10 years ago by people who refuse to learn a language with proper dynamic allocation |
| [19:27:07] | wagnerrp: | :) |
| [19:27:34] | Beirdo: | hehe, how about COBOL that was autoconverted to Ada that you have to now rewrite in proper Ada? |
| [19:27:46] | Beirdo: | that was a fun year and a half of my life. |
| [19:30:04] | danielk22: | wagnerrp: I had to convert some of Ken Perlin's stuff to production. It was always in C. No matter what the language it was written in. I never convinced him to use revision control. I did learn a lot about how to make quick prototypes and test different variations from him though. |
| [19:33:28] | stuartm: | Beirdo: thanks, I've been a little busy but if I can find the time I'd like to run some test transcodes in addition to reviewing, especially since a bad transcode can destroy a recording with no mechanism to recover the original |
| [19:34:05] | Beirdo: | yeah, there is a setting to not overwrite the original though |
| [19:34:09] | Beirdo: | but sure thing |
| [19:34:17] | stuartm: | it's not enabled by default though |
| [19:35:34] | stuartm: | it doesn't hurt to be extra cautious IMHO |
| [19:35:43] | Beirdo: | agreed |
| [19:36:17] | Beirdo: | well, it can wait a bit to get put in, let me know when you feel it's ready |
| [19:37:14] | Beirdo: | better to put it in right than fast |
| [19:38:39] | Beirdo: | seemed to work great on my recordings, and I have a few test recordings from Justin to run through tonight |
| [19:49:54] | danielk22: | wagnerrp: are you a rocket scientist? :) |
| [19:52:00] | wagnerrp: | aye |
| [19:52:16] | wagnerrp: | CFD actually, turbomachinery |
| [19:52:45] | wagnerrp: | the code in question was some flow solver |
| [19:54:15] | wagnerrp: | we just have far too much fortran in the aerospace industry, and its not old code thats floating around, its new stuff |
| [19:54:56] | wagnerrp: | the problem isnt that fortran is bad or slow, per se |
| [19:55:13] | wagnerrp: | more that it only lends itself to purpose written tools |
| [19:55:30] | wagnerrp: | its a lot harder to make things modular than in C or C++ |
| [19:55:55] | Beirdo: | sounds like a case of job security for the Fortran programmers |
| [19:56:47] | Beirdo: | oh look. meeting, followed by interview. Blech. So much for getting work done this afternoon |
| [19:57:21] | wagnerrp: | you cant easily add in another equation to the set, or apply a different turbulence model, or a different order solver, or different flux limiters, or... |
| [19:57:35] | wagnerrp: | c++ can do classes, c can do function pointers |
| [19:57:52] | wagnerrp: | in fortran, you have to go directly into the code and add another conditional |
| [19:58:28] | wagnerrp: | oh and namelists... i /hate/ namelists |
| [19:58:55] | wagnerrp: | and for that matter, any form of IO in general |
| [20:07:05] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 250 seconds) | |
| [20:07:59] | stuartm: | agh, backend's deadlocked again |
| [20:08:18] | stuartm: | why does that always happen when I don't have the time to debug? |
| [20:10:08] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [20:16:02] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has quit () | |
| [20:20:34] | danielk22: | stuartm: just send it a kill -6 and keep the backtrace for later.. |
| [20:20:46] | danielk22: | s/backtrace/core |
| [20:27:16] | stuartm: | the obvious tends to elude me when I'm trying to save recordings from failing :) I wouldn't say I'm good under pressure ;) |
| [20:29:41] | danielk22: | wagnerrp: I was forced to use Ada @ NYU. I read the FORTRAN spec once just to know what people were complaining about, but never had to actually deal with it. |
| [20:32:49] | danielk22: | wagnerrp: at least you never have marketing tell you you should do 6 months of work in a week because it's not "rocket science" :) |
| [20:35:00] | wagnerrp: | my complaint is just that there is no free-form IO |
| [20:35:07] | wagnerrp: | its all structured, even binary reads and writes |
| [20:35:44] | wagnerrp: | ever write is prepended and appended by a block denoting how much was written |
| [20:36:07] | wagnerrp: | meaning you can only read things in in exactly the same order you wrote them out |
| [20:36:46] | wagnerrp: | and all text input must be specially formatted as per namelist rules, which look similar to windows ini files |
| [20:40:09] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [20:45:57] | sphery: | danielk22: want me to revert the mysql_opt_reconnect change, even before we have a "redo statement stuff after reconnect" patch? |
| [20:46:56] | sphery: | seems it's not helping--and taking it back out would allow people to continue down the paths they've already gotten used to for living with the issues/lockups |
| [21:14:15] | brfransen (brfransen!~brfransen@216.254.250.47) has quit (Read error: Connection reset by peer) | |
| [21:14:41] | brfransen (brfransen!~brfransen@216.254.250.47) has joined #mythtv | |
| [21:22:47] | brfransen (brfransen!~brfransen@216.254.250.47) has quit (Read error: Connection reset by peer) | |
| [21:26:07] | brfransen (brfransen!~brfransen@216.254.250.47) has joined #mythtv | |
| [21:55:13] | Kunalagon (Kunalagon!~Kunalagon@212.200.241.18) has joined #mythtv | |
| [21:56:18] | Kunalagon (Kunalagon!~Kunalagon@212.200.241.18) has quit (Client Quit) | |
| [22:01:05] | Gibby (Gibby!~Gibby@204.118.10.244) has quit (Quit: ZNC - http://znc.sourceforge.net) | |
| [22:13:22] | j-rod is now known as j-rod|afk | |
| [23:02:31] | knightr (knightr!~knightr@mythtv/developer/knightr) has quit (Ping timeout: 250 seconds) | |
| [23:07:19] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [23:14:05] | knightr (knightr!~knightr@mythtv/developer/knightr) has joined #mythtv | |
| [23:19:38] | andreax1 (andreax1!~andreaz@p57B94FE8.dip.t-dialin.net) has quit (Read error: Connection reset by peer) | |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.