Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

pmtime needs to have AppNap disabled on OSX #20

Open
tallpsmith opened this Issue May 12, 2015 · 12 comments

Comments

Projects
None yet
3 participants
Contributor

tallpsmith commented May 12, 2015

PCP on OSX once install shows choppy pmcharts on Laptops, even when plugged into power, because the OSX AppNap feature which is enabled by default seems to want to sleep the pmtime process periodically.

I used to be able to use the "Get Info" action on the pmtime application bundle and disable AppNap, but in OSX Yosemite it doesn't seem to be there any more.

Ideally as part of the OSX packaging, the pmtime/pmchart are effectively marked correctly to disable AppNap.

Contributor

natoscott commented May 12, 2015

Thanks Paul - this may be the way to go ... https://www.ihash.eu/2015/01/disable-app-nap-per-app-os-x-yosemite/

It'd be good to figure how to disable this out-of-the-box ... maybe using these hints:
https://jamfnation.jamfsoftware.com/discussion.html?id=10566

Contributor

natoscott commented May 13, 2015

Hey Paul,

I've put a pcp-3.10.5 prerelease dmg on ftp.pcp.io which is built with the plist key change suggested in the second URL above - lemme know how it goes?

Contributor

tallpsmith commented May 13, 2015

while I can see the PLIST key in there, unfortunately pmtime still shows up with "yes" in the App Nap column in the energy view.. odd. (see attached image)

activity_monitor__applications_in_last_8_hours__and_pmtime_needs_to_have_appnap_disabled_on_osx_ issue__20 _performancecopilot_pcp

Contributor

natoscott commented May 13, 2015

Hmm, odd indeed - is it unexpected that pmchart is Yes but pmtime No in that column? (was that changed "by hand"?)

Contributor

tallpsmith commented May 13, 2015

I didn't and can't find out how to change it anymore. No longer shows up in
the application "get info" option like it used to (for any app )
On Wed, 13 May 2015 at 13:42 Nathan Scott notifications@github.com wrote:

Hmm, odd indeed - is it unexpected that pmchart is Yes but pmtime No in
that column? (was that changed "by hand"?)


Reply to this email directly or view it on GitHub
#20 (comment)
.

Contributor

natoscott commented May 13, 2015

Some of the notes I read on the interwebs suggested that the property may be set, but not showing up in some tools ... can you confirm that the problem definitely still happens with that latest dmg?

Thanks Paul.

it's definitely running the latest 3.10.5 prerelease thingy.

On 13 May 2015, at 2:32 pm, Nathan Scott notifications@github.com wrote:

Some of the notes I read on the interwebs suggested that the property may be set, but not showing up in some tools ... can you confirm that the problem definitely still happens with that latest dmg?

Thanks Paul.


Reply to this email directly or view it on GitHub #20 (comment).


pcp mailing list
pcp@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/pcp

natoscott added a commit that referenced this issue May 14, 2015

build: update mac plist properties, attempt an appnap workaround
Possible App Nap workaround (issue #20) as suggested by the 'net.
May or may not work on latest OSX - but its just a plist property
so no harm done even if it doesn't help.
Contributor

tallpsmith commented May 31, 2015

posting here some stuff I put on IRC:

There is some discussion that the PLIST key needed is not what we thought:

https://discussions.apple.com/thread/6641221?start=0&tstart=0

And should be LSAppNapIsDisabled instead of NSAppSleepDisabled however I've tried modifying the pmtime PLIST file to use this key and it does not seem to make any difference.

The only reliabel way so far is to use Terminal to run:

defaults write io.pcp.pmtime NSAppSleepDisabled -bool YES

Contributor

natoscott commented Jun 1, 2015

Hey Paul,

As discussed on IRC, I've put a pcp-3.10.5-0-prerelease2.dmg below ftp.pcp.io/projects/pcp/download/mac to try out. It has both the install script approach and the alternate plist file update in place, so hopefully this works!

cheers.

Contributor

tallpsmith commented Jun 1, 2015

I downloaded prerelease2 and installed it, but it's still choppy. I notice too that the property doesn't look 'set' from the terminal, though I'm not clear whether it's supposed to be visible in this way:

[ ~ ]$ defaults read io.pcp.pmtime 2015-06-02 08:28:41.674 defaults[77662:687283] Domain /Users/psmith/Library/Preferences/io.pcp.pmtime does not exist

pmtime still has a Yes in the App Nap column of the Activity Viewer, which is the dreaded sign that it's going to still be affected.

Contributor

natoscott commented Jun 1, 2015

| /Users/psmith/Library/Preferences/io.pcp.pmtime

... looks user-specific, whereas the plist change we want to be system-wide for all users I guess...?

Contributor

tallpsmith commented Jun 1, 2015

ideally it should be set system wide I think yes.

I think the error above is just when there is no locally defined preferences yet for that class, and it's ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment