Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Speed up launching extension from VSCode debugging panel and other launch.json changes #478

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

lilleyse
Copy link
Contributor

@lilleyse lilleyse commented Oct 3, 2023

I made a few tweaks to launch.json

  • Disabled symbol loading for everything except libcesium.omniverse.plugin.so. This improves start time from 70 seconds to 15 seconds on Linux.
    • The one exception is Development App (Kit Debug) where we still include all symbols
    • This required switching to cppdgb and using symbolLoadInfo
    • No change for Windows, which was already fast enough
  • Removed a few configurations
    • Performance Tracing - I'm concerned the VSCode debugger may affect performance numbers even when running release builds, so this configuration has been removed and should be run on the command line instead
    • Performance Testing App - same as above
    • C++ Unit Tests - these will eventually be folded into the tests extension, in the meantime they can still be run on the command line
    • Tests Extension - requires Ensure main extension starts before tests extension in launch configuration #470 to be fixed first
  • Switched back to using VSCode integrated terminal. The original reason for switching to an external terminal is no longer applicable: Change launch.json files to use external terminal #371.

@lilleyse lilleyse merged commit 088c916 into main Oct 3, 2023
3 checks passed
@lilleyse lilleyse deleted the update-launch-json branch October 3, 2023 20:01
@lilleyse lilleyse mentioned this pull request Oct 20, 2023
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