A.N.T Landscape add-on broken after psutil API change #63982

Closed
opened 2019-04-29 14:21:12 +02:00 by Predrag Ivanović · 8 comments

System Information
Operating system: CRUX Linux
Graphics card: GT640

Blender Version
Broken: distribution build, using system Python
Worked: the official buildbot build

Short description of error
Due to a API change in psutil, A.N.T Landscape can't be enabled.

Exact steps for others to reproduce the error
When Blender is built to use system Python, and psutil is installed,
A.N.T Landscape throws a following error when I try to enable it:

<snip>
AttributeError: 'Process' object has no attribute 'get_cpu_times'
<snip>

Full log ant-log.txt

I asked in #blendercoders, and bzztploink@ suggested a proper way to fix it,
and I came up with this:
0001-ant-landscape-Fix-for-psutil-API-change.patch

I am not a developer, so I am not sure how 'proper' it actually is (my fault, not his :) ),
but it seems to work, tested with a few builds so far.

The reason Blender's Python works fine is that, as far as I can tell, psutil is not shipped with it.
Also, I am not sure when did psutil upstream changes occur, I had the same issue with an old version and the
latest one.

**System Information** Operating system: CRUX Linux Graphics card: GT640 **Blender Version** Broken: distribution build, using system Python Worked: the official buildbot build **Short description of error** Due to a API change in psutil, A.N.T Landscape can't be enabled. **Exact steps for others to reproduce the error** When Blender is built to use system Python, and psutil is installed, A.N.T Landscape throws a following error when I try to enable it: ``` <snip> AttributeError: 'Process' object has no attribute 'get_cpu_times' <snip> ``` Full log [ant-log.txt](https://archive.blender.org/developer/F6991637/ant-log.txt) I asked in #blendercoders, and bzztploink@ suggested a proper way to fix it, and I came up with this: [0001-ant-landscape-Fix-for-psutil-API-change.patch](https://archive.blender.org/developer/F6991660/0001-ant-landscape-Fix-for-psutil-API-change.patch) I am not a developer, so I am not sure how 'proper' it actually is (my fault, not his :) ), but it seems to work, tested with a few builds so far. The reason Blender's Python works fine is that, as far as I can tell, psutil is not shipped with it. Also, I am not sure when did psutil upstream changes occur, I had the same issue with an old version and the latest one.

Added subscriber: @Pedja

Added subscriber: @Pedja
Member

Added subscriber: @lichtwerk

Added subscriber: @lichtwerk
Philipp Oeser self-assigned this 2019-04-29 15:45:00 +02:00
Member

Sounds reasonable, will doublecheck....

Sounds reasonable, will doublecheck....

In #63982#667797, @lichtwerk wrote:
Sounds reasonable, will doublecheck....

For reference, patch is at https://developer.blender.org/D4759

> In #63982#667797, @lichtwerk wrote: > Sounds reasonable, will doublecheck.... For reference, patch is at https://developer.blender.org/D4759
Philipp Oeser removed their assignment 2019-04-30 08:56:53 +02:00
Jacques Lucke was assigned by Philipp Oeser 2019-04-30 08:56:53 +02:00
Member

Added subscriber: @JacquesLucke

Added subscriber: @JacquesLucke
Member

Assigning to @JacquesLucke then (because he did the review)

Assigning to @JacquesLucke then (because he did the review)

This issue was referenced by 89568c1a42

This issue was referenced by 89568c1a42eb894d4c4b51dbc0a896f20a64bcfe
Member

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#63982
No description provided.