Another place to look for possible speed improvements

[BrettB]

Sub,

Things are working pretty well for me with 2.1.5 now. So, I'm moving on to asking for "would be nice" items.

One of the places where we've noticed a "lagging" is when deleting recordings. As an example, I just deleted a Big Brother recording. I've attached the mvp.log from this example. If I'm reading it correctly, the only logging is when I click on the "OK" in the confirmation popup at 9:50:59.976 and then at 9:51:06.646 for the UiStatic.Dispose() (which I think is after the deletion is completed and the confirmation box is removed):

Code:

2011-07-17 09:50:59.876    [DEBUG][3]    User pressed: Return
2011-07-17 09:51:06.646    [DEBUG][3]    UiStatic.Dispose()

So it took about 7 seconds. That wouldn't be soo bad, but there is no feedback to the user that the "OK" was actually received and is being processed. Since sometimes the remote commands don't get transmitted/received for whatever reasons, the user doesn't really know if it's just taking a while, or the button press got missed. Then the user (me, in this case) tries pressing "OK" again. This click gets queued up, and once the delete is finished and the display goes back to the recordings list, then the "OK" gets processed, and up pops a details display for another recording:

Code:

2011-07-17 09:51:07.083    [DEBUG][3]    User pressed: Return
2011-07-17 09:51:07.083    [DEBUG][3]    ShowDetailsPopup()
2011-07-17 09:51:07.114    [DEBUG][3]    PlaybackPositionHelper.GetPlaybackPosition(E:\NPVR Recordings\Star Trek The Next Generation\Star Trek The Next Generation_20110707_04000500.ts)
2011-07-17 09:51:07.130    [DEBUG][3]    PlaybackPositionHelper.GetLastPlaybackDate(E:\NPVR Recordings\Star Trek The Next Generation\Star Trek The Next Generation_20110707_04000500.ts)
2011-07-17 09:51:07.130    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonNormal"]
2011-07-17 09:51:07.130    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonSelected"]
2011-07-17 09:51:07.130    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonNormal"]
2011-07-17 09:51:07.130    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonSelected"]
2011-07-17 09:51:07.130    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonNormal"]
2011-07-17 09:51:07.130    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonSelected"]
2011-07-17 09:51:07.130    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonNormal"]
2011-07-17 09:51:07.130    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonSelected"]
2011-07-17 09:51:07.130    [DEBUG][3]    OnKeyDown() complete

I don't know if there is anything that you can do to speed up the deletion. If you'd like me to do a test with a version that will log more in that process, I can. However, if you could at least add some visual feedback that it is "working" that would be much appreciated. Maybe you could clear out the contents of the pop-up window, but still leave it displayed? Or close the pop-up window, but leave the recordings list window empty until it's reloaded and ready to go?

[mvallevand]

As an example, I just deleted a Big Brother recording.

Wise!

Martin

[johnsonx42]

There are lots of places in NPVR that could use a "working..." or "please wait..." pop-up, or some other sort of visual indication that the remote key was received and action is being taken.

[mvallevand]

I hope there won't be too many popups, the Recording Sort popup just seems in the way IMO. I watch the background list anyway since the popup text doesn't always match what I see on screen.

Martin

[BrettB]

I hope there won't be too many popups, the Recording Sort popup just seems in the way IMO.

I agree with Martin, that I don't really want to add something new (which will just slow things down more). BUT, if we can have some sort of visual cue that the keypress has been received and is working that is going to happen anyway (such as closing the popup?)--just at the beginning instead of at the end--then it wouldn't slow things down more and it would give that visual feedback.

I don't know if something like this is possible or not, but I thought that I would offer it as a suggestion in case it is.

[BrettB]

I agree with Martin, that I don't really want to add something new (which will just slow things down more). BUT, if we can have some sort of visual cue that the keypress has been received and is working that is going to happen anyway (such as closing the popup?)--just at the beginning instead of at the end--then it wouldn't slow things down more and it would give that visual feedback.

I don't know if something like this is possible or not, but I thought that I would offer it as a suggestion in case it is.

OK, now I'm really getting into territory that I only understand a little about... Another idea that I just had was maybe the alpha value (?) of the popup could be changed to make it darker (sort of grayed out) and the rendering of that command could be handled by the NMT (or video card) in parallel while NPVR was continuing and doing the actual deletion?

[sub]

Sub,

Things are working pretty well for me with 2.1.5 now. So, I'm moving on to asking for "would be nice" items.

One of the places where we've noticed a "lagging" is when deleting recordings. As an example, I just deleted a Big Brother recording. I've attached the mvp.log from this example. If I'm reading it correctly, the only logging is when I click on the "OK" in the confirmation popup at 9:50:59.976 and then at 9:51:06.646 for the UiStatic.Dispose() (which I think is after the deletion is completed and the confirmation box is removed

Can you post a log that shows you going into the Recording screen?

[BrettB]

Can you post a log that shows you going into the Recording screen?

Here you go. To make it nice and clean, I just started a new NMT session, went into the Recordings screen, and then exited.

[sub]

Can you reproduce the slow delete using this patch and repost the log?

[BrettB]

Here's another log where I select one of the recordings to get the details popup screen and then exit.

It does have a couple of sets of errors about missing elements:

Code:

2011-07-17 12:16:08.639    [DEBUG][3]    Element not found: /Skin/Element[@name="ButtonList"]
2011-07-17 12:16:10.698    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonNormal"]
2011-07-17 12:16:10.699    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonSelected"]
2011-07-17 12:16:10.699    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonNormal"]
2011-07-17 12:16:10.700    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonSelected"]
2011-07-17 12:16:10.700    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonNormal"]
2011-07-17 12:16:10.701    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonSelected"]
2011-07-17 12:16:10.701    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonNormal"]
2011-07-17 12:16:10.702    [DEBUG][3]    Element not found: /Skin/Element[@name="PopupLeftButtonSelected"]

But I'm just using the default skin, so I wouldn't think that they should really be any problem.