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

Translator should trigger downstream mining job clean #799

Closed
wants to merge 1 commit into from

Conversation

adoerr
Copy link

@adoerr adoerr commented Mar 19, 2024

Translator should trigger downstream mining job clean up for Extended Mining Jobs.

Please see #797 for more context.

Fixes #797

… Mining jobs

Signed-off-by: adoerr <0xad@gmx.net>
Copy link
Contributor

🐰Bencher

ReportTue, March 19, 2024 at 12:31:00 UTC
ProjectStratum v2 (SRI)
Branch799/merge
Testbedsv1
BenchmarkLatencyLatency Results
nanoseconds (ns) | (Δ%)
Latency Upper Boundary
nanoseconds (ns) | (%)
client-submit-serialize✅ (view plot)6924.900 (+0.72%)7308.643 (94.75%)
client-submit-serialize-deserialize✅ (view plot)7762.600 (-0.29%)8256.024 (94.02%)
client-submit-serialize-deserialize-handle/client-submit-serialize-deserialize-handle✅ (view plot)8422.600 (+0.44%)8835.712 (95.32%)
client-sv1-authorize-serialize-deserialize-handle/client-sv1-authorize-serialize-deserialize-handle✅ (view plot)892.720 (-0.30%)924.520 (96.56%)
client-sv1-authorize-serialize-deserialize/client-sv1-authorize-serialize-deserialize✅ (view plot)691.100 (-0.74%)715.711 (96.56%)
client-sv1-authorize-serialize/client-sv1-authorize-serialize✅ (view plot)247.930 (+0.46%)251.451 (98.60%)
client-sv1-get-authorize/client-sv1-get-authorize✅ (view plot)156.400 (-0.33%)158.926 (98.41%)
client-sv1-get-submit✅ (view plot)6492.000 (-2.21%)7043.922 (92.16%)
client-sv1-get-subscribe/client-sv1-get-subscribe✅ (view plot)277.870 (-0.92%)294.914 (94.22%)
client-sv1-subscribe-serialize-deserialize-handle/client-sv1-subscribe-serialize-deserialize-handle✅ (view plot)736.060 (-1.27%)764.160 (96.32%)
client-sv1-subscribe-serialize-deserialize/client-sv1-subscribe-serialize-deserialize✅ (view plot)601.420 (-1.78%)630.400 (95.40%)
client-sv1-subscribe-serialize/client-sv1-subscribe-serialize✅ (view plot)205.490 (-0.35%)217.605 (94.43%)

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help

Copy link
Contributor

🐰Bencher

ReportTue, March 19, 2024 at 12:31:01 UTC
ProjectStratum v2 (SRI)
Branchfix-issue-797
Testbedsv2
BenchmarkLatencyLatency Results
nanoseconds (ns) | (Δ%)
Latency Upper Boundary
nanoseconds (ns) | (%)
client_sv2_handle_message_common✅ (view plot)44.782 (+0.71%)45.461 (98.51%)
client_sv2_handle_message_mining✅ (view plot)73.962 (+6.61%)79.441 (93.10%)
client_sv2_mining_message_submit_standard✅ (view plot)14.654 (-0.06%)14.702 (99.67%)
client_sv2_mining_message_submit_standard_serialize✅ (view plot)266.740 (+0.30%)282.896 (94.29%)
client_sv2_mining_message_submit_standard_serialize_deserialize✅ (view plot)592.490 (+1.48%)605.484 (97.85%)
client_sv2_open_channel✅ (view plot)169.670 (+2.56%)171.125 (99.15%)
client_sv2_open_channel_serialize✅ (view plot)279.810 (-2.20%)298.730 (93.67%)
client_sv2_open_channel_serialize_deserialize✅ (view plot)370.760 (-2.53%)425.366 (87.16%)
client_sv2_setup_connection✅ (view plot)161.730 (-1.70%)179.974 (89.86%)
client_sv2_setup_connection_serialize✅ (view plot)484.160 (+1.75%)501.961 (96.45%)
client_sv2_setup_connection_serialize_deserialize✅ (view plot)927.080 (-5.59%)1069.687 (86.67%)

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help

Copy link
Contributor

🐰Bencher

ReportTue, March 19, 2024 at 12:31:01 UTC
ProjectStratum v2 (SRI)
Branchfix-issue-797
Testbedsv2
BenchmarkEstimated CyclesEstimated Cycles Results
estimated cycles | (Δ%)
Estimated Cycles Upper Boundary
estimated cycles | (%)
InstructionsInstructions Results
instructions | (Δ%)
Instructions Upper Boundary
instructions | (%)
L1 AccessesL1 Accesses Results
accesses | (Δ%)
L1 Accesses Upper Boundary
accesses | (%)
L2 AccessesL2 Accesses Results
accesses | (Δ%)
L2 Accesses Upper Boundary
accesses | (%)
RAM AccessesRAM Accesses Results
accesses | (Δ%)
RAM Accesses Upper Boundary
accesses | (%)
client_sv2_handle_message_common✅ (view plot)2060.000 (+1.36%)2135.001 (96.49%)✅ (view plot)469.000 (+0.75%)486.322 (96.44%)✅ (view plot)730.000 (+0.58%)757.634 (96.35%)✅ (view plot)7.000 (+12.00%)10.185 (68.73%)✅ (view plot)37.000 (+1.54%)39.188 (94.42%)
client_sv2_handle_message_mining✅ (view plot)8134.000 (+0.22%)8205.878 (99.12%)✅ (view plot)2133.000 (+1.32%)2167.973 (98.39%)✅ (view plot)3159.000 (+1.56%)3216.152 (98.22%)✅ (view plot)36.000 (-9.86%)46.516 (77.39%)✅ (view plot)137.000 (-0.23%)139.915 (97.92%)
client_sv2_mining_message_submit_standard✅ (view plot)6296.000 (-0.06%)6481.357 (97.14%)✅ (view plot)1750.000 (+0.14%)1767.497 (99.01%)✅ (view plot)2551.000 (+0.08%)2578.656 (98.93%)✅ (view plot)21.000 (+12.37%)24.179 (86.85%)✅ (view plot)104.000 (-0.48%)109.401 (95.06%)
client_sv2_mining_message_submit_standard_serialize✅ (view plot)14743.000 (+0.67%)14859.263 (99.22%)✅ (view plot)4694.000 (+0.05%)4711.497 (99.63%)✅ (view plot)6753.000 (-0.02%)6783.906 (99.54%)✅ (view plot)51.000 (+5.70%)54.484 (93.60%)✅ (view plot)221.000 (+1.12%)224.348 (98.51%)
client_sv2_mining_message_submit_standard_serialize_deserialize✅ (view plot)27324.000 (+0.13%)27516.889 (99.30%)✅ (view plot)10537.000 (+0.02%)10554.497 (99.83%)✅ (view plot)15334.000 (-0.00%)15363.538 (99.81%)✅ (view plot)88.000 (+4.14%)90.121 (97.65%)✅ (view plot)330.000 (+0.15%)335.810 (98.27%)
client_sv2_open_channel✅ (view plot)4447.000 (-0.36%)4611.710 (96.43%)✅ (view plot)1461.000 (+0.24%)1478.322 (98.83%)✅ (view plot)2157.000 (+0.30%)2178.741 (99.00%)✅ (view plot)10.000 (-9.09%)14.895 (67.14%)✅ (view plot)64.000 (-0.78%)68.229 (93.80%)
client_sv2_open_channel_serialize✅ (view plot)14160.000 (+0.45%)14281.631 (99.15%)✅ (view plot)5064.000 (+0.07%)5081.322 (99.66%)✅ (view plot)7320.000 (+0.05%)7347.475 (99.63%)✅ (view plot)38.000 (-0.33%)42.613 (89.17%)✅ (view plot)190.000 (+0.93%)193.433 (98.23%)
client_sv2_open_channel_serialize_deserialize✅ (view plot)22406.000 (-0.25%)22701.494 (98.70%)✅ (view plot)7979.000 (+0.04%)7996.322 (99.78%)✅ (view plot)11616.000 (+0.05%)11642.696 (99.77%)✅ (view plot)72.000 (+0.88%)79.821 (90.20%)✅ (view plot)298.000 (-0.63%)305.923 (97.41%)
client_sv2_setup_connection✅ (view plot)4741.000 (+0.64%)4811.620 (98.53%)✅ (view plot)1502.000 (+0.23%)1519.322 (98.86%)✅ (view plot)2276.000 (+0.23%)2303.281 (98.82%)✅ (view plot)10.000 (-10.11%)15.246 (65.59%)✅ (view plot)69.000 (+1.28%)70.747 (97.53%)
client_sv2_setup_connection_serialize✅ (view plot)16274.000 (+0.76%)16328.435 (99.67%)✅ (view plot)5963.000 (+0.06%)5980.322 (99.71%)✅ (view plot)8654.000 (-0.01%)8688.308 (99.61%)✅ (view plot)47.000 (+6.06%)49.569 (94.82%)✅ (view plot)211.000 (+1.50%)212.895 (99.11%)
client_sv2_setup_connection_serialize_deserialize✅ (view plot)35518.000 (+0.28%)35558.409 (99.89%)✅ (view plot)14806.000 (+0.02%)14823.322 (99.88%)✅ (view plot)21743.000 (-0.03%)21782.222 (99.82%)✅ (view plot)102.000 (+10.49%)102.072 (99.93%)✅ (view plot)379.000 (+0.43%)381.340 (99.39%)

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help

Copy link
Contributor

🐰Bencher

ReportTue, March 19, 2024 at 12:31:06 UTC
ProjectStratum v2 (SRI)
Branchfix-issue-797
Testbedsv1
BenchmarkEstimated CyclesEstimated Cycles Results
estimated cycles | (Δ%)
Estimated Cycles Upper Boundary
estimated cycles | (%)
InstructionsInstructions Results
instructions | (Δ%)
Instructions Upper Boundary
instructions | (%)
L1 AccessesL1 Accesses Results
accesses | (Δ%)
L1 Accesses Upper Boundary
accesses | (%)
L2 AccessesL2 Accesses Results
accesses | (Δ%)
L2 Accesses Upper Boundary
accesses | (%)
RAM AccessesRAM Accesses Results
accesses | (Δ%)
RAM Accesses Upper Boundary
accesses | (%)
get_authorize✅ (view plot)8474.000 (+0.20%)8514.460 (99.52%)✅ (view plot)3746.000 (-0.13%)3773.693 (99.27%)✅ (view plot)5249.000 (-0.19%)5298.704 (99.06%)✅ (view plot)8.000 (+14.29%)9.754 (82.02%)✅ (view plot)91.000 (+0.69%)91.940 (98.98%)
get_submit✅ (view plot)95579.000 (+0.03%)95628.648 (99.95%)✅ (view plot)59439.000 (-0.03%)59527.406 (99.85%)✅ (view plot)85349.000 (-0.04%)85509.987 (99.81%)✅ (view plot)58.000 (+8.67%)58.818 (98.61%)✅ (view plot)284.000 (+0.40%)285.615 (99.43%)
get_subscribe✅ (view plot)8045.000 (+0.65%)8083.214 (99.53%)✅ (view plot)2841.000 (-0.05%)2848.456 (99.74%)✅ (view plot)3965.000 (-0.18%)3983.374 (99.54%)🚨 (view plot | view alert)18.000 (+23.61%)17.638 (102.05%)✅ (view plot)114.000 (+1.05%)115.226 (98.94%)
serialize_authorize✅ (view plot)12193.000 (-0.23%)12363.431 (98.62%)✅ (view plot)5317.000 (-0.09%)5344.693 (99.48%)✅ (view plot)7413.000 (-0.11%)7458.618 (99.39%)✅ (view plot)11.000 (+4.14%)13.425 (81.93%)✅ (view plot)135.000 (-0.46%)138.792 (97.27%)
serialize_deserialize_authorize✅ (view plot)24514.000 (+0.05%)24596.218 (99.67%)✅ (view plot)9898.000 (-0.10%)9953.387 (99.44%)✅ (view plot)13954.000 (-0.15%)14056.965 (99.27%)✅ (view plot)40.000 (+9.03%)41.698 (95.93%)✅ (view plot)296.000 (+0.15%)297.661 (99.44%)
serialize_deserialize_handle_authorize✅ (view plot)30233.000 (+0.17%)30282.853 (99.84%)✅ (view plot)12101.000 (-0.04%)12128.693 (99.77%)✅ (view plot)17118.000 (-0.07%)17173.736 (99.68%)✅ (view plot)61.000 (+5.06%)63.597 (95.92%)✅ (view plot)366.000 (+0.39%)367.739 (99.53%)
serialize_deserialize_handle_submit✅ (view plot)126438.000 (+0.01%)126510.549 (99.94%)✅ (view plot)73224.000 (-0.02%)73312.406 (99.88%)✅ (view plot)104938.000 (-0.04%)105101.703 (99.84%)🚨 (view plot | view alert)128.000 (+9.58%)126.057 (101.54%)✅ (view plot)596.000 (-0.05%)599.362 (99.44%)
serialize_deserialize_handle_subscribe✅ (view plot)27557.000 (+0.28%)27580.627 (99.91%)✅ (view plot)9643.000 (-0.01%)9650.456 (99.92%)✅ (view plot)13632.000 (-0.07%)13654.891 (99.83%)✅ (view plot)69.000 (+8.13%)70.661 (97.65%)✅ (view plot)388.000 (+0.44%)389.464 (99.62%)
serialize_deserialize_submit✅ (view plot)115027.000 (-0.04%)115248.411 (99.81%)✅ (view plot)68001.000 (-0.05%)68177.812 (99.74%)✅ (view plot)97552.000 (-0.06%)97860.703 (99.68%)✅ (view plot)72.000 (+6.86%)72.202 (99.72%)✅ (view plot)489.000 (-0.05%)491.826 (99.43%)
serialize_deserialize_subscribe✅ (view plot)22946.000 (+0.18%)22962.539 (99.93%)✅ (view plot)8195.000 (-0.03%)8209.912 (99.82%)✅ (view plot)11536.000 (-0.08%)11569.930 (99.71%)✅ (view plot)42.000 (+7.52%)46.014 (91.28%)✅ (view plot)320.000 (+0.31%)320.778 (99.76%)
serialize_submit✅ (view plot)99862.000 (-0.02%)100043.247 (99.82%)✅ (view plot)61483.000 (-0.03%)61571.406 (99.86%)✅ (view plot)88197.000 (-0.03%)88352.002 (99.82%)✅ (view plot)58.000 (+6.91%)58.342 (99.41%)✅ (view plot)325.000 (-0.04%)327.493 (99.24%)
serialize_subscribe✅ (view plot)11340.000 (+0.06%)11477.631 (98.80%)✅ (view plot)4188.000 (-0.03%)4195.456 (99.82%)✅ (view plot)5825.000 (-0.09%)5838.305 (99.77%)🚨 (view plot | view alert)18.000 (+18.03%)17.826 (100.98%)✅ (view plot)155.000 (-0.04%)158.914 (97.54%)

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help

@adoerr
Copy link
Author

adoerr commented Mar 19, 2024

Closing the PR for now because it needs a more detailed discussion and perhaps some infrastructure support.

Discussion will continue as part of issue #797

@adoerr adoerr closed this Mar 19, 2024
@adoerr adoerr deleted the fix-issue-797 branch March 19, 2024 20:56
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.

Translator should trigger downstream mining job clean-up for Extended Mining Jobs
1 participant