More guidance in dynamic circuits pg re: buffer overflow error #2846
Conversation
|
One or more of the following people are relevant to this code:
|
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
| " - Central controller: Memory usage scales proportionally with the number of measurements used by the central controller for control flow." | ||
| " - Central controller: Memory usage scales proportionally with the number of measurements used by the central controller for control flow.\n", | ||
| "\n", | ||
| "<Admonition type=\"note\" title=\"Buffer overflow error\">\n", |
There was a problem hiding this comment.
Does this need to be a note? It's ultimately a limit on the measurement buffer, similar to other memory limits.
There was a problem hiding this comment.
Ah I see - so no need to bring it out with any more emphasis than the others on the page? I misunderstood and thought it was an addendum to the very last one specifically. I'll adjust it!
Honestly, I was confused by that yesterday myself, trying to find the page in the toc. I don't know why this is the case but I'll ask around to see if anyone knows a good reason - otherwise we'll put them together! |
|
Ready for re-review - I've added links, moved the two limits pages together under Manage jobs, and added the buffer overflow content as a bullet: 
|
|
The new changes LGTM, thanks for this! |
cc: @henryzou50