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

Log records count computing is optimized #718

Closed
wants to merge 5 commits into from

Conversation

demvlad
Copy link
Contributor

@demvlad demvlad commented Mar 14, 2024

The log records count is calculated during log loading and saved in the log stats data.
This count value is used in spectrum graph directly.

This comment has been minimized.

Copy link

sonarcloud bot commented Mar 14, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@demvlad
Copy link
Contributor Author

demvlad commented Mar 14, 2024

@haslinghuis
Thank's.
Yes, it was wrong maximum limit. Resolved.

Copy link

Do you want to test this code? Here you have an automated build:
Betaflight-Blackbox-Explorer-Linux
Betaflight-Blackbox-Explorer-macOS
Betaflight-Blackbox-Explorer-Windows
WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

@nerdCopter
Copy link
Member

The NWjs build is obsolete. Closing this PR. It can be re-opened against new branch 3.7.0_NWjs_maintenance for those of very few us who want a "final" installable; however, after a few weeks±, non-PWA PR's will not be accepted.

@nerdCopter nerdCopter closed this Apr 4, 2024
@demvlad demvlad deleted the log_rec_count_improvement branch May 14, 2024 15:20
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.

None yet

3 participants