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

[issue#137]: Add mechanism for reset all flop of Sequential #302

Merged
merged 21 commits into from
May 18, 2023

Conversation

RPG-coder-intc
Copy link
Contributor

@RPG-coder-intc RPG-coder-intc commented Mar 7, 2023

Related Issue(s)

Issue #137

Fix #137

Testing

  • TBA

Backwards-compatibility

  • This will modify Sequential class
  • This feature will be modified to make the effects backward compatible

Documentation

  • TBA

lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
@RPG-coder-intc RPG-coder-intc marked this pull request as ready for review March 16, 2023 13:59
Copy link
Contributor

@mkorbel1 mkorbel1 left a comment

Choose a reason for hiding this comment

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

Looking on the right path!

lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
test/conditionals_test.dart Outdated Show resolved Hide resolved
test/conditionals_test.dart Outdated Show resolved Hide resolved
test/counter_wintf_test.dart Outdated Show resolved Hide resolved
Copy link
Contributor

@mkorbel1 mkorbel1 left a comment

Choose a reason for hiding this comment

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

Nice work! Some suggestions for improvement

lib/src/modules/conditional.dart Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Outdated Show resolved Hide resolved
test/counter_wintf_test.dart Outdated Show resolved Hide resolved
lib/src/modules/conditional.dart Show resolved Hide resolved
@@ -32,7 +34,9 @@ class CounterInterface extends Interface<CounterDirection> {

class Counter extends Module {
late final CounterInterface intf;
Counter(CounterInterface intf) {
late bool resetRoot;
Copy link
Contributor

Choose a reason for hiding this comment

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

i think you're not using this resetRoot anymore, delete?

Copy link
Contributor

@mkorbel1 mkorbel1 left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you for the contribution!

@mkorbel1
Copy link
Contributor

Fix #137

@mkorbel1 mkorbel1 merged commit 63b0dd2 into intel:main May 18, 2023
2 checks passed
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.

Add a mechanism to reset all flops of a Sequential
3 participants