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

BENB scale endless velocity to its previous range #145

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

benblaise-intech
Copy link
Contributor

No description provided.

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

Change in average Code Health of affected files: +0.00 (8.28 -> 8.28)

View detailed results in CodeScene

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

Change in average Code Health of affected files: +0.00 (8.28 -> 8.28)

View detailed results in CodeScene

@Greg-Orca
Copy link

Greg-Orca commented Jan 29, 2025

Test Result:

This PR fixes the Endless issue: https://github.com/orgs/intechstudio/projects/3/views/1?pane=issue&itemId=94759870

The Endless Velocity is consistent.

minor bug:
At Velocity = 0 and Sensitivity = 1, no values are sent.
(At Velocity = 0 and Sensitivity = 2, values are sent as expected.)

@benblaise-intech benblaise-intech self-assigned this Jan 30, 2025
Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

Change in average Code Health of affected files: +0.00 (8.28 -> 8.28)

View detailed results in CodeScene

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

Change in average Code Health of affected files: +0.30 (8.28 -> 8.58)

  • Improving Code Health: 4 findings(s) ✅

View detailed results in CodeScene

@@ -98,18 +98,20 @@ void grid_ui_element_endless_page_change_cb(struct grid_ui_element* ele, uint8_t
// }
}

uint8_t grid_ui_endless_update_trigger(struct grid_ui_element* ele, int stabilized, int16_t delta, uint64_t* endless_last_real_time, double* delta_vel_frac) {
uint8_t grid_ui_endless_update_trigger(struct grid_ui_element* ele, int stabilized, int16_t delta, uint64_t* endless_last_real_time, double* delta_frac) {

Choose a reason for hiding this comment

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

✅ Getting better: Complex Method
grid_ui_endless_update_trigger decreases in cyclomatic complexity from 21 to 20, threshold = 9

Comment on lines -344 to -347
if (update) {
old_value->phase_a = new_value->phase_a;
old_value->phase_b = new_value->phase_b;
}

Choose a reason for hiding this comment

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

✅ No longer an issue: Complex Method
grid_ui_endless_store_input is no longer above the threshold for cyclomatic complexity

Comment on lines -344 to -347
if (update) {
old_value->phase_a = new_value->phase_a;
old_value->phase_b = new_value->phase_b;
}

Choose a reason for hiding this comment

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

✅ No longer an issue: Bumpy Road Ahead
grid_ui_endless_store_input is no longer above the threshold for logical blocks with deeply nested code

@@ -98,18 +98,20 @@ void grid_ui_element_endless_page_change_cb(struct grid_ui_element* ele, uint8_t
// }
}

uint8_t grid_ui_endless_update_trigger(struct grid_ui_element* ele, int stabilized, int16_t delta, uint64_t* endless_last_real_time, double* delta_vel_frac) {
uint8_t grid_ui_endless_update_trigger(struct grid_ui_element* ele, int stabilized, int16_t delta, uint64_t* endless_last_real_time, double* delta_frac) {

Choose a reason for hiding this comment

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

✅ Getting better: Overall Code Complexity
The mean cyclomatic complexity decreases from 6.43 to 6.14, threshold = 4

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

Change in average Code Health of affected files: +0.30 (8.28 -> 8.58)

  • Improving Code Health: 4 findings(s) ✅

View detailed results in CodeScene

@SukuWc SukuWc marked this pull request as ready for review February 5, 2025 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready for QA
Development

Successfully merging this pull request may close these issues.

2 participants