Skip to content

Conversation

@ManuelCarullo
Copy link

Usdmanager can be very slow when opening heavy usd files with lots of data. This is still the case when syntax highlighting and link parsing is turned off.
I narrowed this down to the setHtml() call on the TextBrowser that is taking most of the time.

With this PR I suggest adding a Raw View mode, that uses the TextEditor in read only mode instead of the TextBrowser, which leads to significant performance improvements, when loading heavy usd files.

I'm not sure how actively this repo is still maintained, so sorry if this is the wrong place for the PR.
Also you can ignore the changes in docs/conf.py and docs/requirements.txt, just wanted to match my documentation build to the official one.

raw_view_button raw_view_enabled raw_view_preferences
  • Add Raw View mode for improved performance
  • Use TextEditor instead of TextBrowser for Raw View mode to avoid HTML processing overhead.
  • Disable syntax highlighting and link parsing in Raw View
  • Add preference to default new tabs to Raw View mode
  • Update documentation and configuration
  • Add Raw View mode documentation to usage.md
  • Add Raw View shortcut (Ctrl+Shift+V) to keyboard shortcuts
  • Fix Sphinx configuration for Python 3 compatibility
  • Modernize conf.py

* Add Raw View mode for improved performance

- Use TextEditor instead of TextBrowser for Raw View mode to avoid HTML processing overhead.
- Disable syntax highlighting and link parsing in Raw View
- Add preference to default new tabs to Raw View mode

* Update documentation and configuration

- Add Raw View mode documentation to usage.md
- Add Raw View shortcut (Ctrl+Shift+V) to keyboard shortcuts
- Fix Sphinx configuration for Python 3 compatibility
- Modernize conf.py with proper version import and theme

Signed-off-by: Manuel Carullo <[email protected]>
@tracypriest
Copy link
Collaborator

@ManuelCarullo Thanks for the update! It looks like we don't have a CLA for you. Could you sign the contributor agreement and email that to [email protected]? Thanks!

@ManuelCarullo
Copy link
Author

@ManuelCarullo Thanks for the update! It looks like we don't have a CLA for you. Could you sign the contributor agreement and email that to [email protected]? Thanks!

Oh right, totally forgot about that sorry. I just sent it out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants