Differential D6457
Update USD Error Handling Authored by superfunc on Dec 20 2019, 5:45 AM. Tags Subscribers None
Details As listed in https://developer.blender.org/T72204, and its aforementioned issue, this is a small patch to catch a more comprehensive set of errors that may be emitted in the process of constructing a USD stage. Note: This work is done solely in a personal capacity and is not endorsed by any of my current or previous employers.
Diff Detail
Event TimelineComment Actions I'm not really impressed with the errors produced by the USD library. When it cannot find the plugin JSON files, this just shows "USD Export: internal issue (Failed verification: ' fileFormat ')" in the GUI, which won't mean anything to artists. The error code string for this error is "TF_DIAGNOSTIC_CODING_ERROR_TYPE", which also doesn't help if we want to translate the error message to something that is more user-friendly. @superfunc do you know whether the other error cases have a more specific identifier? Subsequent WM_reportf() overwrite the message in the status bar, so in effect this will show only the last message to the user (unless they have the Info editor open, which isn't open by default). Probably it's better to just printf() the low-level USD error, and keep original message in place unless we can extract something more explanatory. Comment Actions This is a really good point. Sorry, I was thinking a bit too technical forgetting the audience of this.
Let me look into something nicer we can surface for this.
Agreed for this. Comment Actions @Sybren A. Stüvel (sybren) Sorry again for the hastiness of that first change. I think a better approach here is to handle these known cases explicitly to craft I am submitting a new diff that handles a couple cases.
Beyond those, we relay that its an unknown issue and we log to the console as suggested, rather than the user facing area.
| ||||||
