You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In fixing #1880, I noticed that many of our notebooks (e.g. estimation-hardcoded-circuit, estimation-random-circuit and others) don't render resource estimation details using the widget, but just directly using a pattern such as:
result=qsharp.estimate("RunHardcodedCircuit()")
result
This shows a table where the markdown isn't rendered correctly even with the linked fix in place (as that only applies to widgets), e.g.
In investigating the root cause for this, there are two issues to address:
Firstly, we have the below code in our pip package, but nothing in our docs or samples mentions requiring these packages, and no error or output is displayed to tell users to install them. We should document or provide output on this requirement if needed.
try:
# Both markdown and mdx_math (from python-markdown-math) must be present for our markdown# rendering logic to work. If either is missing, we'll fall back to plain text.importmarkdownimportmdx_mathhas_markdown=TrueexceptImportError:
has_markdown=False
Secondly, even installing those packages didn't resolve the markdown rending in the table as shown in the screen shot above, so that needs to be fixed anyway.
In fixing #1880, I noticed that many of our notebooks (e.g.
estimation-hardcoded-circuit
,estimation-random-circuit
and others) don't render resource estimation details using the widget, but just directly using a pattern such as:This shows a table where the markdown isn't rendered correctly even with the linked fix in place (as that only applies to widgets), e.g.
In investigating the root cause for this, there are two issues to address:
Firstly, we have the below code in our pip package, but nothing in our docs or samples mentions requiring these packages, and no error or output is displayed to tell users to install them. We should document or provide output on this requirement if needed.
Secondly, even installing those packages didn't resolve the markdown rending in the table as shown in the screen shot above, so that needs to be fixed anyway.
CC @msoeken and @ivanbasov
The text was updated successfully, but these errors were encountered: