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.

Sign up for GitHub

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

Jump to bottom

update scipy notebook #368

Merged
merged 2 commits into from
Sep 29, 2023
Merged

update scipy notebook #368

merged 2 commits into from
Sep 29, 2023

Conversation

chrbeckm
Copy link
Member

Closes #363

jpwgnr
jpwgnr requested changes Sep 29, 2023
View reviewed changes
python/scientific-python.ipynb Outdated
@@ -58,7 +58,9 @@
"# physical constants\n",
"import scipy.constants as const\n",
"\n",
"const.epsilon_0"
"planck = const.Planck # returns the value\n",
"planck2 = const.physical_constants[\"Planck constant\"] # returns value, unit and uncertainty\n",
Copy link

Choose a reason for hiding this comment

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

Give better name, e.g. planck_full

python/scientific-python.ipynb Outdated
"const.epsilon_0"
"planck = const.Planck # returns the value\n",
"planck2 = const.physical_constants[\"Planck constant\"] # returns value, unit and uncertainty\n",
"print(f\"{planck = }\", f\"{planck2 = }\", sep=\"\\n\")"
Copy link

Choose a reason for hiding this comment

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

Change here correspondingly

@jpwgnr
Copy link

jpwgnr commented Sep 29, 2023 •
edited
Loading

Also add the other points regarding plots mentioned by @aknierim in #363

@chrbeckm chrbeckm merged commit 1fabcc4 into main Sep 29, 2023
1 check passed
@chrbeckm chrbeckm deleted the update_scipy_notebook branch September 29, 2023 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aknierim aknierim aknierim approved these changes

@jpwgnr jpwgnr jpwgnr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Scipy notebook
3 participants
@chrbeckm @jpwgnr @aknierim