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

Claudiaregio/jupyter read me #430

Merged
merged 11 commits into from
Nov 11, 2020
Merged

Claudiaregio/jupyter read me #430

merged 11 commits into from
Nov 11, 2020

Conversation

claudiaregio
Copy link
Contributor

Thinking about removing feature details section because of their heavy tie to Python, looking for feedback.

@claudiaregio claudiaregio requested a review from a team as a code owner November 10, 2020 01:10
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@DonJayamanne DonJayamanne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't agree with the title. 99% of users would want to use interactive window and old notebooks. Going into market place gives them the false impression that this extension works only in insisted as the title suggests.
When extensions are blogged about a screenshot of readme page is used and this will add to this confusion.

I would instead add a separate section just for native notebooks, which happens to be only a feature of this extension, and indicate the requirements in that section.

README.md Outdated

## Set up your environment
- **Step 2.** Install the [Jupyter Extension](https://code.visualstudio.com/docs/python/python-tutorial#_prerequisites) in VS Code Insiders - CHANGE ME
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is CHANGE ME a todo here? Just waiting for the link?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes just waiting for the marketplace extension link here

README.md Outdated Show resolved Hide resolved
<!-- use less words -->
- **Step 2.1** If you are interested in using Python, we recommend you install the [Python Extension](https://marketplace.visualstudio.com/items?itemName=ms-python.python) instead.
- **Step 3.** If not working with Python, make sure you have a kernel specification that corresponds to the language you would like to use installed on Jupyter on your machine.
- **Step 4.** Open or create a notebook file and start coding!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mentioning the command to create a new blank notebook might be helpful here. We've seen users confused by this before and they can't just create a new empty one like a .py file.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh, you mention that just below, probably not needed here then.

README.md Outdated
- Select your Python interpreter by clicking on the status bar

<img src=https://raw.githubusercontent.com/microsoft/vscode-jupyter/main/images/InterpreterSelectionZoom.gif width=280 height=100>
## Set up your environment
Copy link
Member

@IanMatthewHuff IanMatthewHuff Nov 10, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This section header name doesn't really seem to apply anymore. Getting started? Starting a notebook? Running a notebook? Not fully sure on the best name.

README.md Outdated

- Configure the debugger through the Debug Activity Bar
<!-- use less words -->
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another todo here?


- Configure tests by running the `Configure Tests` command
<img src=https://raw.githubusercontent.com/microsoft/vscode-jupyter/main/images/Jupyter%20README/CreateNewNotebook.png>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The old GIFs specifically set width and height. I would think that's probably to have them aligned correctly. Do these look ok if you shrink / expand the window size?

Copy link
Contributor Author

@claudiaregio claudiaregio Nov 10, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edited: The default sizes came from the Python read me, I need to see how these render before changing the sizes back.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated

To see all available Python commands, open the Command Palette and type `Python`.
To see all available Jupyter Notebook commands, open the Command Palette and type `Jupyter` or `Notebook`.

## Feature details

Learn more about the rich features of the Jupyter extension:

- [IntelliSense](https://code.visualstudio.com/docs/python/editing#_autocomplete-and-intellisense): Edit your code with auto-completion, code navigation, syntax checking and more
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Little bit unsure of how we list this support. We do support this for Python with the extension installed, and in a limited fashion for other languages. But we certainly don't have something like code navigation for Julia or other languages. Same thing with our listing of debugging support.

README.md Outdated Show resolved Hide resolved
Copy link
Member

@IanMatthewHuff IanMatthewHuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There were a few changes that I think we need here. Probably of biggest importance was that we're now advertising as a multi-language extension, but we still have lots of features listed in the debugging / formatting / intellisense sections that we don't directly provide and are only available for Python + the Python Extension.

Copy link
Member

@IanMatthewHuff IanMatthewHuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good to me. GIFs don't have the sizing, but not sure if that is actually an issue or not.

@rchiodo rchiodo merged commit 68bee5c into main Nov 11, 2020
@rchiodo rchiodo deleted the claudiaregio/jupyterReadMe branch November 11, 2020 17:46
rchiodo pushed a commit that referenced this pull request Nov 11, 2020
* remove refactoring from description

* Updated Jupyter ReadMe

* Update README.md

Co-authored-by: Joyce Er <[email protected]>

* Update README.md

Co-authored-by: Joyce Er <[email protected]>

* Update README.md

Co-authored-by: Joyce Er <[email protected]>

* Edit step 3 in setup

* Jupyter ReadMe 2

* plz let this be the last jupyter readme update

* updated jupyter extension marketplace link

Co-authored-by: Joyce Er <[email protected]>
rchiodo added a commit that referenced this pull request Nov 11, 2020
* Claudiaregio/jupyter read me (#430)

* remove refactoring from description

* Updated Jupyter ReadMe

* Update README.md

Co-authored-by: Joyce Er <[email protected]>

* Update README.md

Co-authored-by: Joyce Er <[email protected]>

* Update README.md

Co-authored-by: Joyce Er <[email protected]>

* Edit step 3 in setup

* Jupyter ReadMe 2

* plz let this be the last jupyter readme update

* updated jupyter extension marketplace link

Co-authored-by: Joyce Er <[email protected]>

* Final update for release

Co-authored-by: claudiaregio <[email protected]>
Co-authored-by: Joyce Er <[email protected]>
rchiodo added a commit that referenced this pull request Nov 11, 2020
* Claudiaregio/jupyter read me (#430)

* remove refactoring from description

* Updated Jupyter ReadMe

* Update README.md

Co-authored-by: Joyce Er <[email protected]>

* Update README.md

Co-authored-by: Joyce Er <[email protected]>

* Update README.md

Co-authored-by: Joyce Er <[email protected]>

* Edit step 3 in setup

* Jupyter ReadMe 2

* plz let this be the last jupyter readme update

* updated jupyter extension marketplace link

Co-authored-by: Joyce Er <[email protected]>

* Add new license for extension

Co-authored-by: claudiaregio <[email protected]>
Co-authored-by: Joyce Er <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants