{"action":"created","issue":{"url":"https://api.github.com/repos/PixarAnimationStudios/USD/issues/549","repository_url":"https://api.github.com/repos/PixarAnimationStudios/USD","labels_url":"https://api.github.com/repos/PixarAnimationStudios/USD/issues/549/labels{/name}","comments_url":"https://api.github.com/repos/PixarAnimationStudios/USD/issues/549/comments","events_url":"https://api.github.com/repos/PixarAnimationStudios/USD/issues/549/events","html_url":"https://github.com/PixarAnimationStudios/USD/pull/549","id":339163907,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk5OTA3NDEx","number":549,"title":"Allow this library to build against Houdini 17","user":{"login":"marktucker","id":1513015,"node_id":"MDQ6VXNlcjE1MTMwMTU=","avatar_url":"https://avatars2.githubusercontent.com/u/1513015?v=4","gravatar_id":"","url":"https://api.github.com/users/marktucker","html_url":"https://github.com/marktucker","followers_url":"https://api.github.com/users/marktucker/followers","following_url":"https://api.github.com/users/marktucker/following{/other_user}","gists_url":"https://api.github.com/users/marktucker/gists{/gist_id}","starred_url":"https://api.github.com/users/marktucker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marktucker/subscriptions","organizations_url":"https://api.github.com/users/marktucker/orgs","repos_url":"https://api.github.com/users/marktucker/repos","events_url":"https://api.github.com/users/marktucker/events{/privacy}","received_events_url":"https://api.github.com/users/marktucker/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-07-07T17:57:17Z","updated_at":"2018-07-07T18:29:41Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/PixarAnimationStudios/USD/pulls/549","html_url":"https://github.com/PixarAnimationStudios/USD/pull/549","diff_url":"https://github.com/PixarAnimationStudios/USD/pull/549.diff","patch_url":"https://github.com/PixarAnimationStudios/USD/pull/549.patch"},"body":"Trying to bring this library in line with the needs of Houdini 17:\r\n* Several changes to packed primitive function signatures\r\n* Support float16 UT_Quaternions and vectors\r\n* Exporting a few symbols and classes needed by H17 plugins. Removing\r\n  export claim from GusdDefaultArray which is a header-only template class\r\n  and so shouldn't claim to export any symbols\r\n* GEO_IOTranslator accepts usd, usda, or usdc extensions\r\n* GEO_IOTranslator loads proxy and default prims, rather than just proxy\r\n* USD_CustomTraverse intialization invoked by an explicit function call\r\n  (which is called from gustInitialize)\r\n* USD_XformCache env var initialization fix, most likely peculiar to the\r\n  way that H17 causes this library to be loaded\r\n* Convert uses of HDK_API_VERSION to SYS_VERSION_FULL_INT. This latter\r\n  variable is a more reliable reflection of the Houdini build version,\r\n  and so should be used in preference to HDK_API_VERSION\r\n* Use GA and GT constants for Houdini geometry attribute names where\r\n  applicable. Use GT_Offset constants instead of assuming GT_Offset==int\r\n* In printf functions, use SYS_PRI constants for reliable cross-platform\r\n  printing of 64 bit values\r\n* More error checking when converting Houdini attributes to/from USD\r\n  attributes, and when calculating prim xforms"},"comment":{"url":"https://api.github.com/repos/PixarAnimationStudios/USD/issues/comments/403234775","html_url":"https://github.com/PixarAnimationStudios/USD/pull/549#issuecomment-403234775","issue_url":"https://api.github.com/repos/PixarAnimationStudios/USD/issues/549","id":403234775,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMzIzNDc3NQ==","user":{"login":"spiffmon","id":6220392,"node_id":"MDQ6VXNlcjYyMjAzOTI=","avatar_url":"https://avatars2.githubusercontent.com/u/6220392?v=4","gravatar_id":"","url":"https://api.github.com/users/spiffmon","html_url":"https://github.com/spiffmon","followers_url":"https://api.github.com/users/spiffmon/followers","following_url":"https://api.github.com/users/spiffmon/following{/other_user}","gists_url":"https://api.github.com/users/spiffmon/gists{/gist_id}","starred_url":"https://api.github.com/users/spiffmon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spiffmon/subscriptions","organizations_url":"https://api.github.com/users/spiffmon/orgs","repos_url":"https://api.github.com/users/spiffmon/repos","events_url":"https://api.github.com/users/spiffmon/events{/privacy}","received_events_url":"https://api.github.com/users/spiffmon/received_events","type":"User","site_admin":false},"created_at":"2018-07-07T18:29:41Z","updated_at":"2018-07-07T18:29:41Z","author_association":"MEMBER","body":"Sweet - thank you, Mark!\n\n--spiff\n\n\nOn Sat, Jul 7, 2018 at 10:57 AM Mark Tucker <notifications@github.com>\nwrote:\n\n> Trying to bring this library in line with the needs of Houdini 17:\n>\n>    - Several changes to packed primitive function signatures\n>    - Support float16 UT_Quaternions and vectors\n>    - Exporting a few symbols and classes needed by H17 plugins. Removing\n>    export claim from GusdDefaultArray which is a header-only template\n>    class\n>    and so shouldn't claim to export any symbols\n>    - GEO_IOTranslator accepts usd, usda, or usdc extensions\n>    - GEO_IOTranslator loads proxy and default prims, rather than just\n>    proxy\n>    - USD_CustomTraverse intialization invoked by an explicit function call\n>    (which is called from gustInitialize)\n>    - USD_XformCache env var initialization fix, most likely peculiar to\n>    the\n>    way that H17 causes this library to be loaded\n>    - Convert uses of HDK_API_VERSION to SYS_VERSION_FULL_INT. This latter\n>    variable is a more reliable reflection of the Houdini build version,\n>    and so should be used in preference to HDK_API_VERSION\n>    - Use GA and GT constants for Houdini geometry attribute names where\n>    applicable. Use GT_Offset constants instead of assuming GT_Offset==int\n>    - In printf functions, use SYS_PRI constants for reliable\n>    cross-platform\n>    printing of 64 bit values\n>    - More error checking when converting Houdini attributes to/from USD\n>    attributes, and when calculating prim xforms\n>\n> ------------------------------\n> You can view, comment on, or merge this pull request online at:\n>\n>   https://github.com/PixarAnimationStudios/USD/pull/549\n> Commit Summary\n>\n>    - A number of changes required for building this library for use in\n>    Houdini\n>    - Fixes to last commit to make gusd compilable against Houdini 16.5\n>    and 16.0.\n>\n> File Changes\n>\n>    - *M* third_party/houdini/lib/gusd/GEO_IOTranslator.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-0>\n>    (9)\n>    - *M* third_party/houdini/lib/gusd/GT_PackedUSD.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-1>\n>    (12)\n>    - *M* third_party/houdini/lib/gusd/GT_PackedUSD.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-2>\n>    (2)\n>    - *M* third_party/houdini/lib/gusd/GT_PointInstancer.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-3>\n>    (4)\n>    - *M* third_party/houdini/lib/gusd/GT_PrimCache.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-4>\n>    (10)\n>    - *M* third_party/houdini/lib/gusd/GT_Utils.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-5>\n>    (8)\n>    - *M* third_party/houdini/lib/gusd/GT_Utils.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-6>\n>    (4)\n>    - *M* third_party/houdini/lib/gusd/GT_VtArray.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-7>\n>    (7)\n>    - *M* third_party/houdini/lib/gusd/GU_PackedUSD.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-8>\n>    (69)\n>    - *M* third_party/houdini/lib/gusd/GU_PackedUSD.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-9>\n>    (41)\n>    - *M* third_party/houdini/lib/gusd/GU_USD.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-10>\n>    (13)\n>    - *M* third_party/houdini/lib/gusd/PRM_Shared.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-11>\n>    (5)\n>    - *M* third_party/houdini/lib/gusd/USD_CustomTraverse.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-12>\n>    (12)\n>    - *M* third_party/houdini/lib/gusd/USD_CustomTraverse.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-13>\n>    (2)\n>    - *M* third_party/houdini/lib/gusd/USD_XformCache.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-14>\n>    (5)\n>    - *M* third_party/houdini/lib/gusd/UT_Gf.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-15>\n>    (55)\n>    - *M* third_party/houdini/lib/gusd/UT_TypeTraits.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-16>\n>    (14)\n>    - *M* third_party/houdini/lib/gusd/defaultArray.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-17>\n>    (2)\n>    - *M* third_party/houdini/lib/gusd/instancerWrapper.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-18>\n>    (5)\n>    - *M* third_party/houdini/lib/gusd/meshWrapper.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-19>\n>    (7)\n>    - *M* third_party/houdini/lib/gusd/plugin.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-20>\n>    (8)\n>    - *M* third_party/houdini/lib/gusd/primWrapper.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-21>\n>    (32)\n>    - *M* third_party/houdini/lib/gusd/refiner.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-22>\n>    (6)\n>    - *M* third_party/houdini/lib/gusd/stageCache.cpp\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-23>\n>    (53)\n>    - *M* third_party/houdini/lib/gusd/writeCtrlFlags.h\n>    <https://github.com/PixarAnimationStudios/USD/pull/549/files#diff-24>\n>    (2)\n>\n> Patch Links:\n>\n>    - https://github.com/PixarAnimationStudios/USD/pull/549.patch\n>    - https://github.com/PixarAnimationStudios/USD/pull/549.diff\n>\n> —\n> You are receiving this because you are subscribed to this thread.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/PixarAnimationStudios/USD/pull/549>, or mute the\n> thread\n> <https://github.com/notifications/unsubscribe-auth/AF7qaCd57SGHz7gEP_f2Q4vCaBG9vBUIks5uEPaDgaJpZM4VGcRf>\n> .\n>\n"}}
{"actor":{"display_login":"spiffmon"}}