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

Merge New Sequencer into Develop branch #1658

Merged
merged 142 commits into from
Feb 15, 2023
Merged

Conversation

ToniRamirezM
Copy link
Contributor

Merges New Sequencer into Develop branch

ToniRamirezM and others added 30 commits December 22, 2022 12:20
…erfaces by the finalizer from the dbmanager, and implementing the remaining resourses Sub.

Signed-off-by: Nikolay Nedkov <[email protected]>
Signed-off-by: Nikolay Nedkov <[email protected]>
Signed-off-by: Nikolay Nedkov <[email protected]>
Signed-off-by: Nikolay Nedkov <[email protected]>
* WIP

* WIP

* WIP
Signed-off-by: Nikolay Nedkov <[email protected]>
Signed-off-by: Nikolay Nedkov <[email protected]>
…nalizer

wip: adding some implementation of the finalizer, defining needed int…
…rker

Initial sequencer.worker implementation (WIP)
Signed-off-by: Nikolay Nedkov <[email protected]>
Signed-off-by: Nikolay Nedkov <[email protected]>
Signed-off-by: Nikolay Nedkov <[email protected]>
* wip: adding implementation for .isCurrBatchAboveLimitWindow', and doing small refactor.

Signed-off-by: Nikolay Nedkov <[email protected]>

* WIP

* WIP

* WIP

* WIP

* WIP

* WIP

* WIP

* WIP

* ForceBatch unit test

* SequenceForceBatch unit test

* Fix forcedBatch check

* linter

* Fix reset forcedBatch

* batch and forcedbatch table modified

* fix

* WIP

* Finish dbmanager

* Branches Integration

Signed-off-by: Nikolay Nedkov <[email protected]>
Co-authored-by: Nikolay Nedkov <[email protected]>
Co-authored-by: Alonso <[email protected]>
* Closing Signals Manager first implementation

* Closing Signals Manager first implementation

* Closing Signals Manager first implementation

* refactor
Signed-off-by: Nikolay Nedkov <[email protected]>
Added worker unit testing. Missing TODOs implementation done
Signed-off-by: Nikolay Nedkov <[email protected]>
@ARR552 ARR552 force-pushed the feature/newSequencer branch 3 times, most recently from 7b79a57 to 3ac6de7 Compare February 14, 2023 11:17
@ToniRamirezM ToniRamirezM requested review from Psykepro and removed request for cool-develope February 15, 2023 10:33
Copy link
Contributor

@ARR552 ARR552 left a comment

Choose a reason for hiding this comment

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

Looks good to me xD. What a tiny PR...

@ToniRamirezM ToniRamirezM merged commit 418c66f into develop Feb 15, 2023
@ToniRamirezM ToniRamirezM deleted the feature/newSequencer branch February 15, 2023 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants