Petri net executor example#276
Merged
Merged
Conversation
Codecov Report
@@ Coverage Diff @@
## main #276 +/- ##
=======================================
Coverage 98.59% 98.59%
=======================================
Files 100 100
Lines 1419 1419
=======================================
Hits 1399 1399
Misses 20 20 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Owner
|
Yeah, looks cool! Thank you! |
Contributor
Author
|
The ci output https://github.com/victimsnino/ReactivePlusPlus/actions/runs/3074503201/jobs/4967539918 and https://github.com/victimsnino/ReactivePlusPlus/actions/runs/3074503201/jobs/4967539973 does not give me a clue what to look at (or I am missing it). Do you see it?
Thomas
… On 17 Sep 2022, at 23:05, Aleksey Loginov ***@***.***> wrote:
Yeah, looks cool! Thank you!
Could you address comments from CI to pass it? Then i would merge it
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.
|
Owner
|
You need to unfold Build with Cmake inside this one error: |
Contributor
Author
|
I was unable to easily reproduce this, clang-14 (which is was I had available) didn't give me this error (with the same flags as in the ci-matrix).. However, I think this edit should fix it. |
Contributor
BENCHMARK RESULTS (AUTOGENERATED)ci-ubuntu-clangObservable constructionTable
Observable liftTable
Observable subscribeTable
Observable subscribe #2Table
Observer constructionTable
OnNextTable
Subscriber constructionTable
SubscriptionTable
bufferTable
chains creation testTable
combine_latestTable
concatTable
distinct_until_changedTable
firstTable
foundamental sourcesTable
fromTable
immediate schedulerTable
justTable
lastTable
mapTable
mergeTable
observe_onTable
publish_subject callbacksTable
publish_subject routinesTable
repeatTable
scanTable
single-threaded locksTable
skipTable
switch_on_nextTable
takeTable
take_lastTable
take_untilTable
trampoline schedulerTable
windowTable
with_latest_fromTable
ci-ubuntu-gccObservable constructionTable
Observable liftTable
Observable subscribeTable
Observable subscribe #2Table
Observer constructionTable
OnNextTable
Subscriber constructionTable
SubscriptionTable
bufferTable
chains creation testTable
combine_latestTable
concatTable
distinct_until_changedTable
firstTable
foundamental sourcesTable
fromTable
immediate schedulerTable
justTable
lastTable
mapTable
mergeTable
observe_onTable
publish_subject callbacksTable
publish_subject routinesTable
repeatTable
scanTable
single-threaded locksTable
skipTable
switch_on_nextTable
takeTable
take_lastTable
take_untilTable
trampoline schedulerTable
windowTable
with_latest_fromTable
ci-windowsObservable constructionTable
Observable liftTable
Observable subscribeTable
Observable subscribe #2Table
Observer constructionTable
OnNextTable
Subscriber constructionTable
SubscriptionTable
bufferTable
chains creation testTable
combine_latestTable
concatTable
distinct_until_changedTable
firstTable
foundamental sourcesTable
fromTable
immediate schedulerTable
justTable
lastTable
mapTable
mergeTable
observe_onTable
publish_subject callbacksTable
publish_subject routinesTable
repeatTable
scanTable
single-threaded locksTable
skipTable
switch_on_nextTable
takeTable
take_lastTable
take_untilTable
trampoline schedulerTable
windowTable
with_latest_fromTable
|
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
AlexInLog
approved these changes
Sep 18, 2022
|
Kudos, SonarCloud Quality Gate passed! |
Contributor
Author
|
A very cool! Will check it out.
Thomas
… On 18 Sep 2022, at 09:07, Aleksey Loginov ***@***.***> wrote:
You need to unfold Build with Cmake inside this one
error:
Error: /home/runner/work/ReactivePlusPlus/ReactivePlusPlus/src/examples/petri/petri.cpp:84:26: error: local variable 'marking' will be copied despite being returned by name [-Werror,-Wreturn-std-move]
[363](https://github.com/victimsnino/ReactivePlusPlus/actions/runs/3074503201/jobs/4967539973#step:5:366)
return marking;
[364](https://github.com/victimsnino/ReactivePlusPlus/actions/runs/3074503201/jobs/4967539973#step:5:367)
^~~~~~~
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.








As promised, an, imho fun and simplified example of implementing Petri net execution using reactive streams.
Note that it could be a bit fancier by executing the transitions in a different context - that way new transitions would get queued while the active transition is still running. Once RPP has something similar to RxCpp's
observe_on_event_loopit would also be straightforward to actually execute transitions parallel using Rx-patterns.Let me know if you think this example adds value to your project and if there's something you'd like to see clarified/improved.