-
-
Notifications
You must be signed in to change notification settings - Fork 73
add documentation for Task / Thread occupancy #549
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! I had trouble running it locally - Documenter hangs at "ExpandTemplates: expanding markdown templates" - any ideas why?
@jpsamaroo I think I may have an idea: Did you run the For some reason that hangs for me as well. However, if I start a REPL, activate the docs environment and That also explains why the Documentation CI step timed out. (Just realized that). Could this potentially be an interaction between the Dagger scheduler / the main thread of the REPL / ...? |
Yeah I ran it from the CLI as well. I'm running with the |
@wolthom yeah let's do that for now - I don't have a whole lot of time to investigate this right now, and I don't want to block adding this information to the docs, so let's just use "dumb" code blocks, and maybe you can just include the result from running on your local system? |
@jpsamaroo I have updated the Could you verify if this also works as expected for you? |
Worked great, thank you! |
This PR adds a section to the Task Spawning documentation page to explain the behavior observed in #548.