QNTM-4394 Match Revit CEF intitializtion settings#9058
QNTM-4394 Match Revit CEF intitializtion settings#9058saintentropy merged 3 commits intoDynamoDS:masterfrom
Conversation
|
We are also initializing CEF here: I don't know if that ever happens but it should follow the same pattern. |
|
Ahh good catch! |
|
@saintentropy I am still curious why I can't get three.js to launch in D4R 2018. This was the check I was using for WebGL Anyway, this LGTM. We should keep a close eye on conflicts between multiple CEF reliant apps, especially librarie.js. I also came across this line which is slightly different in package manager UI (which isn't implemented) but it seems the Singapore team consistently disabled the GPU when using CEF. |
|
|
@mjkkirschner 1) Yes we have verified that and 2) no we have noticed specific issues with Library although there are still intermittent issues with Library that seem to occur independent of this settings change. Specifically, the notes in the older commits were about flickering. That has not be observed in our use... perhaps CEF version changes since then have improved that defect. Regardless, whatever we observe with this change is what is currently observed by users in Revit 2018, 2019, 2020. |
|
All Self service tests appears to pass except 1 that is also failing on master |
|
@saintentropy just pinged @smangarole on this. I believe some recent changes by Luke may be causing failures so we are working to get that sorted out |
|
Ok thanks @alfarok I will go ahead and merge and wait for the test to be resolved. |
Purpose
The purpose of this PR is to match Dynamo’s CEF initialization settings with those set by Revit. Currently the delta is that Dynamo disables GPU acceleration. For Revit 2018 and 2019, Revit does not disable GPU acceleration in the CEF context as there are supported Revit addins who require LMV (and threejs). When Dynamo is run in these contexts, Revit’s CEF settings are the first to be set and Dynamo’s preferences (and other subsequent CEF addins) are ignored. Although not every Dynamo session (Sandbox for example) has this situation, disabling GPU rendering in core has the potential to mask bugs that are present for a majority of users. Dynamo core should match Revit’s CEF settings and version and should continue to match settings and versions as they change in future releases.
QNTM-4934
Declarations
Check these if you believe they are true
*.resxfilesReviewers
@mjkkirschner @alfarok
FYIs
@sm6srw @Dewb