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

Fix tutorial04_execution.ipynb #241

Merged
merged 1 commit into from
Apr 20, 2022
Merged

Fix tutorial04_execution.ipynb #241

merged 1 commit into from
Apr 20, 2022

Conversation

mgkwill
Copy link
Contributor

@mgkwill mgkwill commented Apr 20, 2022

Issue Number:

Objective of pull request: Fix tutorial 4, frame [44]

Pull request checklist

Your PR fulfills the following requirements:

  • Issue created that explains the change and why it's needed
  • Tests are part of the PR (for bug fixes / features)
  • Docs reviewed and added / updated if needed (for bug fixes / features)
  • PR conforms to Coding Conventions
  • PR applys BSD 3-clause or LGPL2.1+ Licenses to all code files
  • Lint (flakeheaven lint src/lava tests/) and (bandit -r src/lava/.) pass locally
  • Build tests (pytest) passes locally

Pull request type

Please check your PR type:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation changes
  • Other (please describe):

What is the current behavior?

  • Runtime initialization has changed and this has broken tutorial 4, frame [44].

What is the new behavior?

  • Update Runtime initialization for tutorial 4, frame [44] to get it working again.

Does this introduce a breaking change?

  • Yes
  • No

Supplemental information

Signed-off-by: Marcus G K Williams <Marcus G K Williams [email protected]>
@mgkwill mgkwill merged commit 1881948 into lava-nc:main Apr 20, 2022
@mgkwill mgkwill deleted the fix_tut04 branch April 20, 2022 23:30
monkin77 pushed a commit to monkin77/thesis-lava that referenced this pull request Jul 12, 2024
Signed-off-by: Marcus G K Williams <Marcus G K Williams [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.

1 participant