{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":1461954,"defaultBranch":"main","name":"OpenShadingLanguage","ownerLogin":"AcademySoftwareFoundation","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2011-03-10T02:44:05.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/40807682?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1715728722.0","currentOid":""},"activityList":{"items":[{"before":"e0f7b4e11596e827e239ff566308ebd4725aba50","after":"14548bb4809331907af38fc96365c06398cfd756","ref":"refs/heads/test_vscode_webinterface","pushedAt":"2024-05-14T23:22:05.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"AlexMWells","name":"Alex Wells","path":"/AlexMWells","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/13861316?s=80&v=4"},"commit":{"message":"Added blank line to see behavior of vscode interface Commit and Push","shortMessageHtmlLink":"Added blank line to see behavior of vscode interface Commit and Push"}},{"before":null,"after":"e0f7b4e11596e827e239ff566308ebd4725aba50","ref":"refs/heads/test_vscode_webinterface","pushedAt":"2024-05-14T23:18:42.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"AlexMWells","name":"Alex Wells","path":"/AlexMWells","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/13861316?s=80&v=4"},"commit":{"message":"testing: Update ref output for OptiX execution order variance (#1816)\n\nWas spuriously failing tests only because on GPU, the points\r\ndidn't always execute in the same order.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"testing: Update ref output for OptiX execution order variance (#1816)"}},{"before":"056ba1802e8b310caa8e1c31737b868466766cdf","after":"e0f7b4e11596e827e239ff566308ebd4725aba50","ref":"refs/heads/main","pushedAt":"2024-05-11T00:16:05.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"testing: Update ref output for OptiX execution order variance (#1816)\n\nWas spuriously failing tests only because on GPU, the points\r\ndidn't always execute in the same order.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"testing: Update ref output for OptiX execution order variance (#1816)"}},{"before":"f6b88b9053f28ccf55b2d3438e0c31657c25404d","after":"056ba1802e8b310caa8e1c31737b868466766cdf","ref":"refs/heads/main","pushedAt":"2024-05-09T23:48:01.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Add lazytrace option to avoid combinatoric optixTrace inlining (#1815)\n\nOptiX inlines all calls to optixTrace at module compilation time. Due to OSL's\r\nlazy layer evaluation, if layer A calls layer B M times, and layer B calls\r\nlayer C N times, that can lead to M*N inlines of optixTrace if layer C contains\r\na trace operation.\r\n\r\nIn practice we've observed single trace ops being inlined hundreds of times,\r\nleading to minutes-long shader compilations.\r\n\r\nThis patch adds a new option, lazytrace, to run layers with trace ops\r\nunconditionally at the start of shader evaluation. This costs some potential\r\nperformance in cases where the trace layer would never be evaluated, but\r\nremoves all the compilation penalties that the inlining was presenting.\r\n\r\n---------\r\n\r\nSigned-off-by: Chris Hellmuth ","shortMessageHtmlLink":"Add lazytrace option to avoid combinatoric optixTrace inlining (#1815)"}},{"before":"bba9b353c27179cd101a210a50281ba8fedc8854","after":"f6b88b9053f28ccf55b2d3438e0c31657c25404d","ref":"refs/heads/main","pushedAt":"2024-05-08T23:19:50.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"fix: Fix testrender GPU regression with bad destruction order (#1814)\n\nA few months back, PR #1733 seems to have switched the order that\r\ntestrender destroys the shading system versus the renderer (services).\r\n\r\nThis made some subtle bugs that were only symptomatic for GPU renders,\r\nbut it's because of the destructor order, where the shadingsystem's\r\ndtr still references the renderer, which cannot be destroyed yet.\r\n\r\nThe clue is that the SS's constructor takes the RS pointer as an\r\nargument. The RS, then, must have been constructed before the SS, and\r\ntherefore we should expect it to be a requirement for the RS to\r\noutlast the lifetime of the SS. (Complex objects should be destroyed\r\nin the opposite order that they were constructed, if they contain\r\nreferences to each other.)\r\n\r\nOne code change is needed to avoid the sanitizer errors that the\r\nincorrect change was originally meant to address: clear shaders when\r\nSimpleRayTracer clears.\r\n\r\nSigned-off-by: Larry Gritz \r\n\r\n---------\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"fix: Fix testrender GPU regression with bad destruction order (#1814)"}},{"before":"b859317915261a39e835e1d5497198de12028142","after":"bba9b353c27179cd101a210a50281ba8fedc8854","ref":"refs/heads/main","pushedAt":"2024-05-08T18:13:35.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"docs: fix outdated or wrong repo URLs (#1811)\n\n* Fix references to old OIIO home\r\n* Fix embarrassing release notes generating script botch that\r\n accidentally used OpenImageIO URLs where OSL URLs were needed.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"docs: fix outdated or wrong repo URLs (#1811)"}},{"before":"3ff5d5d91d707d33b50733e83626aa65fb40973f","after":"b859317915261a39e835e1d5497198de12028142","ref":"refs/heads/main","pushedAt":"2024-05-05T04:58:44.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"cleanup: Switch some deprecated TypeDesc names to modern ones (#1806)\n\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"cleanup: Switch some deprecated TypeDesc names to modern ones (#1806)"}},{"before":"37af00a3daadf1a1f4bb48668050ffd95c16c475","after":"8b88ad8873dce046cca794809bcc1298d6f3898d","ref":"refs/heads/dev-1.13","pushedAt":"2024-05-05T03:23:25.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Stage for v1.13.9.0\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"Stage for v1.13.9.0"}},{"before":"eb9b08789c1cbf0eeadc43bf2f8573b31795fc16","after":"3ff5d5d91d707d33b50733e83626aa65fb40973f","ref":"refs/heads/main","pushedAt":"2024-05-04T01:20:56.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"build: Remove remaining traces of boost (#1808)\n\nReplace the last few uses of boost with robin-map (which is already a dependency for OIIO, although it is new for OSL itself) and functions in OIIO.\r\n\r\nSigned-off-by: Chris Kulla ","shortMessageHtmlLink":"build: Remove remaining traces of boost (#1808)"}},{"before":"89ba4196a5a0016a4b6ebaf1728649b57eac0b0b","after":"8b88ad8873dce046cca794809bcc1298d6f3898d","ref":"refs/heads/release","pushedAt":"2024-05-01T20:03:02.000Z","pushType":"push","commitsCount":12,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Stage for v1.13.9.0\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"Stage for v1.13.9.0"}},{"before":"910a967655f70b862af45826feeba964f834c259","after":"37af00a3daadf1a1f4bb48668050ffd95c16c475","ref":"refs/heads/dev-1.13","pushedAt":"2024-04-30T02:14:19.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"cleanup: change OSL_CONSTEXPR14 to constexpr (#1805)\n\nWe've been C++14 minimum for a long time now, we can just say constexpr.\n\nChange a couple of stray OIIO_CONSTEXPR14 occurrences as well.\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"cleanup: change OSL_CONSTEXPR14 to constexpr (#1805)"}},{"before":"b87cceb29ce6cb721317113be1492e32baeddd34","after":"910a967655f70b862af45826feeba964f834c259","ref":"refs/heads/dev-1.13","pushedAt":"2024-04-29T22:44:45.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Suppress leak sanitizer warnings about internals of robin_hash (#1807)\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"Suppress leak sanitizer warnings about internals of robin_hash (#1807)"}},{"before":"ca136d78a7f8f5deff9af68fc8fc925063ba02e9","after":"eb9b08789c1cbf0eeadc43bf2f8573b31795fc16","ref":"refs/heads/main","pushedAt":"2024-04-29T18:02:26.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Suppress leak sanitizer warnings about internals of robin_hash (#1807)\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"Suppress leak sanitizer warnings about internals of robin_hash (#1807)"}},{"before":"bc55bed79570bf6626d7dc807ffcf4e154ff9923","after":"ca136d78a7f8f5deff9af68fc8fc925063ba02e9","ref":"refs/heads/main","pushedAt":"2024-04-29T18:02:13.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"cleanup: change OSL_CONSTEXPR14 to constexpr (#1805)\n\nWe've been C++14 minimum for a long time now, we can just say constexpr.\r\n\r\nChange a couple of stray OIIO_CONSTEXPR14 occurrences as well.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"cleanup: change OSL_CONSTEXPR14 to constexpr (#1805)"}},{"before":"a21ba65a2c6445511c893aaac03fa81823d552b5","after":"bc55bed79570bf6626d7dc807ffcf4e154ff9923","ref":"refs/heads/main","pushedAt":"2024-04-29T18:01:57.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"fix: Fix ifdef typo (#1804)\n\nfix: Fix typo\r\n\r\nThis is the only way that makes any sense -- define OIIO_HIDE_FORMAT\r\nif it ISN'T already defined.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"fix: Fix ifdef typo (#1804)"}},{"before":"89ba4196a5a0016a4b6ebaf1728649b57eac0b0b","after":"b87cceb29ce6cb721317113be1492e32baeddd34","ref":"refs/heads/dev-1.13","pushedAt":"2024-04-29T05:26:20.000Z","pushType":"push","commitsCount":8,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Add full LLVM 18 support (#1803)\n\nSome methods and enums got renamed from LLVM 17 -> 18, etc.\n\nAlso some adjustment to warning messages about version mixing\nbetween clang and llvm.\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"Add full LLVM 18 support (#1803)"}},{"before":"312d79454eb344a8f810fef9651c791fc0e7524c","after":"a21ba65a2c6445511c893aaac03fa81823d552b5","ref":"refs/heads/main","pushedAt":"2024-04-29T00:11:19.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"build: print commit hash of locally built dependencies (#1798)\n\nEspecially for our \"bleeding edge\" test that builds from the current\r\nmaster of several dependencies, sometimes our tests fail not because\r\nof anything we've done, but because of a change in one of the\r\ndependencies. To help track down exactly which change broke the\r\ndependency, make sure our CI build logs print the specific commit hash\r\nof those packages that we built.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"build: print commit hash of locally built dependencies (#1798)"}},{"before":"33f24ec55731cd838ea7eebc5bf1304ad58450ec","after":"312d79454eb344a8f810fef9651c791fc0e7524c","ref":"refs/heads/main","pushedAt":"2024-04-29T00:10:56.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Add full LLVM 18 support (#1803)\n\nSome methods and enums got renamed from LLVM 17 -> 18, etc.\r\n\r\nAlso some adjustment to warning messages about version mixing\r\nbetween clang and llvm.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"Add full LLVM 18 support (#1803)"}},{"before":"633b3d3599c71b9ed45c700b0a522a0fef8ab471","after":"33f24ec55731cd838ea7eebc5bf1304ad58450ec","ref":"refs/heads/main","pushedAt":"2024-04-29T00:10:33.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Remove the only use of boost::thread_specific_ptr (#1802)\n\nRemove the only use of boost::thread_specific_ptr.\r\n\r\nThis had already been marked as deprecated, and there is concensus that the overhead does not outweight the slight inconvenience of creating one object per thread\r\n\r\nSigned-off-by: Chris Kulla ","shortMessageHtmlLink":"Remove the only use of boost::thread_specific_ptr (#1802)"}},{"before":"44fccbdda4f210fd1012e804991aaeb3f8f41a45","after":"633b3d3599c71b9ed45c700b0a522a0fef8ab471","ref":"refs/heads/main","pushedAt":"2024-04-27T17:01:38.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"ci: Lock down to OIIO 2.5 for icc tests (#1799)\n\nBackground: Intel's icc is essentially discontinued and we think all\r\nthe renderers using OSL have moved on to the newer icx, but Alex Wells\r\nstill likes to sometime use icc for its better facilities for some\r\nSIMD code generation sleuthing that he does. So we keep it in our CI\r\nmatrix to esure we don't break it for OSL until it is truly\r\nunavoidable or he doesn't feel like he needs it anymore.\r\n\r\nSome recent changes to OIIO's simd.h, from PR 4187 in that project,\r\nseem to tweak what appears to be a compiler bug in icc -- bugs were\r\nalso noted in OIIO in the course of this PR and some workarounds were\r\nput there just for icc to make OIIO's own tests pass. Well, once that\r\nwent in, OSL's CI tests for the icc case also started failing. It's\r\nonly texture derivatives that are broken, and in a weird wy that\r\nsmells like a compiler icc bug with SIMD generation similar to what we\r\ndiscovered on the OIIO side.\r\n\r\nLike I said, this is a discontinued compiler that no major renderers\r\nare still using, so it doesn't seem worth tracking it down any further\r\nto find a true fix. So let's just lock that one test case down to OIIO\r\n2.5 (the current release) to avoid the changes that will eventually be\r\nin OIIO 2.6/3.0 and hope that we fully retire icc before OIIO 2.5 is\r\nno longer supported by OSL.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"ci: Lock down to OIIO 2.5 for icc tests (#1799)"}},{"before":"b6c8dde7b9d0d848e6882eb312d926192c9d343d","after":"44fccbdda4f210fd1012e804991aaeb3f8f41a45","ref":"refs/heads/main","pushedAt":"2024-04-18T02:44:37.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"build: Propagate cpp version (#1797)\n\nOIIO recently pushed its master to be C++17 minimum. When using a C++17-minimum OIIO, you need to build OSL with C++17 as well. This broke a couple CI cases for OSL.\r\n\r\nThere were a few spots where we were hard-coded to C++14 on the OSL side rather than even using the C++ version specified for the OSL build, so these are some fixes here to address this.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"build: Propagate cpp version (#1797)"}},{"before":"477d9793474b954a43ff379111f122960fe63ff0","after":"b6c8dde7b9d0d848e6882eb312d926192c9d343d","ref":"refs/heads/main","pushedAt":"2024-04-05T15:28:53.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Make isconnected() work with downstream renderer \"connections.\" (#1782)\n\nThe isconnected() returns 2 when used on an output that is connected to another downstream shader node -- this can be used to limit computation to only the outputs used. However, that misses the important case of where the renderer has declared a shader node output as a renderer output, which happens often when using OSL purely for pattern generation purposes. This change makes that case also return 2.\r\n\r\nThe isconnected test has been updated to include testing this functionality.\r\n\r\nSigned-off-by: Stephen Friedman ","shortMessageHtmlLink":"Make isconnected() work with downstream renderer \"connections.\" (#1782)"}},{"before":"a987aa9e8d86f5391ab36a17dfc2fc45d680b492","after":"477d9793474b954a43ff379111f122960fe63ff0","ref":"refs/heads/main","pushedAt":"2024-04-04T19:36:16.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"docs: More detail about supported platforms in INSTALL.md (#1796)\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"docs: More detail about supported platforms in INSTALL.md (#1796)"}},{"before":"7fe43cec6e947e584cecfda426d83a77df5af05a","after":"a987aa9e8d86f5391ab36a17dfc2fc45d680b492","ref":"refs/heads/main","pushedAt":"2024-04-03T00:33:06.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"oslquery: Simplify include and link needs (#1787)\n\nThe (should be) tiny oslquery library was over-including and\r\nover-linking beyond its actual needs.\r\n\r\n* Remove`#include `, which is a lot of setup, type\r\n definitions, and many transitive header includes. Instead, just\r\n include the very small number of headers and types actually needed\r\n by the OSLQuery class definition.\r\n* In particular, this removes includes of OpenEXR/Imath headers and\r\n references to Imath types, the OIIO TextureSystem (!), and other\r\n big things, from what oslquery.h needs to see.\r\n* liboslquery can link against libOpenImageIO_Util (which it uses\r\n primarily for ustring and TypeDesc), it doesn't need the full\r\n libOpenImageIO, since it doesn't read or write image files.\r\n* Remove some link references to Boost which isn't actually needed for\r\n this library.\r\n* This also lightens the load for the Python module for oslquery.\r\n\r\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"oslquery: Simplify include and link needs (#1787)"}},{"before":"5e47ef444e16b5638d257f400d1866e2ef68fdc5","after":"89ba4196a5a0016a4b6ebaf1728649b57eac0b0b","ref":"refs/heads/release","pushedAt":"2024-04-01T18:45:31.000Z","pushType":"push","commitsCount":8,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Stage for v1.13.8.0\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"Stage for v1.13.8.0"}},{"before":"2c7c4f2f3651c4958b71c2da437a38e317222b60","after":"89ba4196a5a0016a4b6ebaf1728649b57eac0b0b","ref":"refs/heads/dev-1.13","pushedAt":"2024-04-01T18:45:17.000Z","pushType":"push","commitsCount":4,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Stage for v1.13.8.0\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"Stage for v1.13.8.0"}},{"before":"b08abb8d033dbdc019ef1c48e6554397f93f9af4","after":"7fe43cec6e947e584cecfda426d83a77df5af05a","ref":"refs/heads/main","pushedAt":"2024-03-28T22:25:55.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"fix: GPU interpolated param initialization (#1791)\n\nCurrently, the GPU will crash when initializing non-string interpolated parameters with default values.\r\n\r\nIt looks like this is a regression from the ustringhash conversion in 12530f1 (#1732). I believe that before the phase 3 commit, we had global variables on GPU that we memcpy'd the defaults values from. Now after their removal, we're memcpy'ing from an illegal address.\r\n\r\nAs a proposed fix I removed the entire memcpy branch of the code and let interpolated parameters fall back to the default code path that stores the values directly. I'm open to instead guarding the removed block with \"&& !gpu\" making the patch have no effect on cpu, but leaned towards having unified code paths. Still, it's not an area of the code I'm very familiar with, any feedback would be appreciated.\r\n\r\nSigned-off-by: Chris Hellmuth ","shortMessageHtmlLink":"fix: GPU interpolated param initialization (#1791)"}},{"before":"b210ab4def2596579ba14ccd679ebafa1798e662","after":"b08abb8d033dbdc019ef1c48e6554397f93f9af4","ref":"refs/heads/main","pushedAt":"2024-03-28T22:23:51.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"Use new OIIO ustringhash decode methods when available (#1792)\n\nThis eliminates some pointless and costly hash -> ustring round trips.\r\n\r\nSigned-off-by: Chris Hellmuth ","shortMessageHtmlLink":"Use new OIIO ustringhash decode methods when available (#1792)"}},{"before":"fefeb5a099c0fe4948f956007cc56a662f76acc9","after":"b210ab4def2596579ba14ccd679ebafa1798e662","ref":"refs/heads/main","pushedAt":"2024-03-25T23:37:59.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"cleanup: Change long-deprecated OIIO macro to new one (#1788)\n\nSigned-off-by: Larry Gritz ","shortMessageHtmlLink":"cleanup: Change long-deprecated OIIO macro to new one (#1788)"}},{"before":"4738a0fdf5987a8a0fc2bc1c13ec0d6918226e29","after":"fefeb5a099c0fe4948f956007cc56a662f76acc9","ref":"refs/heads/main","pushedAt":"2024-03-09T22:34:18.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"lgritz","name":"Larry Gritz","path":"/lgritz","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/504179?s=80&v=4"},"commit":{"message":"fix: Print closure missing error message at compile time instead of run time. (#1781)\n\nPrevent complications on the GPU by printing error messages at compile time instead of generating code to print them at runtime when a closure doesn't exist.\r\n\r\n---------\r\n\r\nSigned-off-by: Stephen Friedman ","shortMessageHtmlLink":"fix: Print closure missing error message at compile time instead of r…"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAESfto3AA","startCursor":null,"endCursor":null}},"title":"Activity · AcademySoftwareFoundation/OpenShadingLanguage"}