Skip to content

Added wait() to refvar unittests#220

Merged
mgkwill merged 1 commit intolava-nc:mainfrom
bamsumit:main
Mar 8, 2022
Merged

Added wait() to refvar unittests#220
mgkwill merged 1 commit intolava-nc:mainfrom
bamsumit:main

Conversation

@bamsumit
Copy link
Copy Markdown
Contributor

@bamsumit bamsumit commented Mar 8, 2022

Issue Number:

Objective of pull request:

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?

  • test_ref_var_ports fails non-deterministically

What is the new behavior?

  • test_ref_var_ports should always pass

Does this introduce a breaking change?

  • Yes
  • No

Supplemental information

@bamsumit bamsumit linked an issue Mar 8, 2022 that may be closed by this pull request
7 tasks
@bamsumit bamsumit requested review from PhilippPlank and mgkwill March 8, 2022 18:29
Copy link
Copy Markdown
Contributor

@PhilippPlank PhilippPlank left a comment

Choose a reason for hiding this comment

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

The files poetry.lock and pyproject.toml should probably be not commited? @mgkwill what do you think?

@mgkwill
Copy link
Copy Markdown
Contributor

mgkwill commented Mar 8, 2022 via email

@mgkwill mgkwill merged commit 22bb29a into lava-nc:main Mar 8, 2022
monkin77 pushed a commit to monkin77/thesis-lava that referenced this pull request Jul 12, 2024
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.

tests/lava/magma/runtime/test_ref_var_ports.py sometimes fails

3 participants