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

Notebook Stdout for R contains missing characters / misinterprets escape codes #171181

Closed
EthanGraber opened this issue Jan 11, 2023 · 3 comments
Closed
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s)

Comments

@EthanGraber
Copy link

I am running a Jupyter notebook with an R kernel.

The stdout from running R commands (specifically, in this scenario, viewing the R documentation) doesn't appear to be handling underline escape codes properly.

Example:
image

Note how the first few lines contain what appear to be missing characters. E.x.: _�D_�i_�a_�m_�e_�t_�e_�r

Here's the proper output for the same command when entered into my terminal:
image

@DonJayamanne DonJayamanne transferred this issue from microsoft/vscode-notebook-renderers Jan 12, 2023
@DonJayamanne
Copy link
Contributor

I can replicate this issue at my end, heres ipynb file]

ipynb
{
"cells": [
{
 "cell_type": "code",
 "execution_count": 1,
 "metadata": {
  "vscode": {
   "languageId": "r"
  }
 },
 "outputs": [
  {
   "name": "stdout",
   "output_type": "stream",
   "text": [
    "trees                 package:datasets                 R Documentation\n",
    "\n",
    "_\bD_\bi_\ba_\bm_\be_\bt_\be_\br, _\bH_\be_\bi_\bg_\bh_\bt _\ba_\bn_\bd _\bV_\bo_\bl_\bu_\bm_\be _\bf_\bo_\br _\bB_\bl_\ba_\bc_\bk _\bC_\bh_\be_\br_\br_\by _\bT_\br_\be_\be_\bs\n",
    "\n",
    "_\bD_\be_\bs_\bc_\br_\bi_\bp_\bt_\bi_\bo_\bn:\n",
    "\n",
    "     This data set provides measurements of the diameter, height and\n",
    "     volume of timber in 31 felled black cherry trees.  Note that the\n",
    "     diameter (in inches) is erroneously labelled Girth in the data. It\n",
    "     is measured at 4 ft 6 in above the ground.\n",
    "\n",
    "_\bU_\bs_\ba_\bg_\be:\n",
    "\n",
    "     trees\n",
    "     \n",
    "_\bF_\bo_\br_\bm_\ba_\bt:\n",
    "\n",
    "     A data frame with 31 observations on 3 variables.\n",
    "\n",
    "       ‘[,1]’  ‘Girth’   numeric  Tree diameter (rather than girth, actually) in inches \n",
    "       ‘[,2]’  ‘Height’  numeric  Height in ft                                          \n",
    "       ‘[,3]’  ‘Volume’  numeric  Volume of timber in cubic ft                          \n",
    "      \n",
    "_\bS_\bo_\bu_\br_\bc_\be:\n",
    "\n",
    "     Ryan, T. A., Joiner, B. L. and Ryan, B. F. (1976) _The Minitab\n",
    "     Student Handbook_.  Duxbury Press.\n",
    "\n",
    "_\bR_\be_\bf_\be_\br_\be_\bn_\bc_\be_\bs:\n",
    "\n",
    "     Atkinson, A. C. (1985) _Plots, Transformations and Regression_.\n",
    "     Oxford University Press.\n",
    "\n",
    "_\bE_\bx_\ba_\bm_\bp_\bl_\be_\bs:\n",
    "\n",
    "     require(stats); require(graphics)\n",
    "     pairs(trees, panel = panel.smooth, main = \"trees data\")\n",
    "     plot(Volume ~ Girth, data = trees, log = \"xy\")\n",
    "     coplot(log(Volume) ~ log(Girth) | Height, data = trees,\n",
    "            panel = panel.smooth)\n",
    "     summary(fm1 <- lm(log(Volume) ~ log(Girth), data = trees))\n",
    "     summary(fm2 <- update(fm1, ~ . + log(Height), data = trees))\n",
    "     step(fm2)\n",
    "     ## i.e., Volume ~= c * Height * Girth^2  seems reasonable\n",
    "     "
   ]
  }
 ],
 "source": [
  "?trees\n"
 ]
}
],
"metadata": {
"kernelspec": {
 "display_name": "R",
 "language": "R",
 "name": "ir"
},
"language_info": {
 "codemirror_mode": "r",
 "file_extension": ".r",
 "mimetype": "text/x-r-source",
 "name": "R",
 "pygments_lexer": "r",
 "version": "4.2.1"
},
"orig_nbformat": 4,
"vscode": {
 "interpreter": {
  "hash": "49fdd20b2b68a4dd8ebcbb061b9c04f8e52f6472af90d6159269a7c8abf5d944"
 }
}
},
"nbformat": 4,
"nbformat_minor": 2
}

@amunger
Copy link
Contributor

amunger commented Jan 12, 2023

likely related to #118833

@rebornix rebornix added notebook-builtin-renderers under-discussion Issue is under discussion for relevance, priority, approach labels Dec 5, 2023
@rebornix rebornix removed their assignment Dec 5, 2023
@amunger amunger added bug Issue identified by VS Code Team member as probable bug and removed under-discussion Issue is under discussion for relevance, priority, approach labels Dec 7, 2023
@rebornix
Copy link
Member

Merging into #118833

@rebornix rebornix added the *duplicate Issue identified as a duplicate of another issue(s) label Dec 13, 2024
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Jan 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug *duplicate Issue identified as a duplicate of another issue(s)
Projects
None yet
Development

No branches or pull requests

4 participants