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

notebooks: beautify advanced_02 #50

Merged
merged 1 commit into from
Jul 11, 2024
Merged

notebooks: beautify advanced_02 #50

merged 1 commit into from
Jul 11, 2024

Conversation

JochenSiegWork
Copy link
Collaborator 

- Fix typos, remove wrong type hints,
  add more explanation

@JochenSiegWork
notebooks: beautify advanced_02
8b06772 
    - Fix typos, remove wrong type hints,
      add more explanation
@JochenSiegWork JochenSiegWork self-assigned this Jul 11, 2024
c-w-feldmann
c-w-feldmann approved these changes Jul 11, 2024
View reviewed changes
Copy link
Collaborator

@c-w-feldmann c-w-feldmann left a comment

Choose a reason for hiding this comment

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

LGTM

@c-w-feldmann c-w-feldmann added type: documentation Improvements or additions to documentation type: maintenance Improvement of code or keeping the code up to date labels Jul 11, 2024
@JochenSiegWork JochenSiegWork merged commit 188edb2 into main Jul 11, 2024
14 checks passed
@JochenSiegWork JochenSiegWork deleted the beautify_advanced_02_notebook branch July 11, 2024 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@c-w-feldmann c-w-feldmann c-w-feldmann approved these changes

Assignees

@JochenSiegWork JochenSiegWork

Labels
type: documentation Improvements or additions to documentation type: maintenance Improvement of code or keeping the code up to date
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JochenSiegWork @c-w-feldmann