search

UDST / pandana

Pandas Network Analysis by UrbanSim: fast accessibility metrics and shortest paths, using contraction hierarchies :world_map:
http://udst.github.io/pandana
GNU Affero General Public License v3.0
385 stars 84 forks source link

Fix build on non-x86 #158

Closed pkubaj closed 3 years ago

pkubaj commented 3 years ago

cpuid is not available outside of amd64/i386.

smmaurer commented 3 years ago

Hi @pkubaj, thanks for contributing this!

We ran into the same issue in #152. My tentative solution was to drop the #include itself, because it doesn't seem like HyperThreading.h is actually used by the current code.

But your solution of refining the #ifdef is more conservative, so I'm inclined to go with this. I'll run a couple of tests and then merge this.