Closed mmcky closed 1 year ago
sphinx.ast
as markdown ansi
cells. So we need to include this output in the output
cell structure. {
"cell_type": "code",
"execution_count": 11,
"id": "e00265c5",
"metadata": {
"hide-output": false
},
"outputs": [
{
"data": {
"text/plain": [
"list"
]
},
"execution_count": 11,
"metadata": {},
"output_type": "execute_result"
}
],
"source": [
"x = ['foo', 'bar']\n",
"type(x)"
]
},
{
"cell_type": "markdown",
"id": "95c912ac",
"metadata": {
"hide-output": false
},
"source": [
"```myst-ansi\n",
"list\n",
"```\n"
]
},
it is currently returning the following in the notebook
{
"cell_type": "code",
"execution_count": null,
"id": "6f44030f",
"metadata": {
"hide-output": false
},
"outputs": [],
"source": [
"x = [10, -10]\n",
"max(x)"
]
},
{
"cell_type": "markdown",
"id": "ab64e956",
"metadata": {
"hide-output": false
},
"source": [
"```myst-ansi\n",
"10\n",
"```\n"
]
},
but this duplicates cell code outputs and it makes testing stochastic given outputs can include memory locations that change etc.
@jstac -- are you happy if notebooks continue to not include execution-outputs
in download notebooks -- or do you think this would be useful to have in the download notebook (pre-executed)?
This PR updates to be compatible with the broader update to
sphinx5
in the Executable Books stack and specifically enables compatibility withmyst-nb>=0.14
as a requirement.