feat: SDKS-2280 Flutter fixed timestamp observer with variable speed generation #222
Conversation
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
- Switch from sliding window (900s) to fixed timestamp approach - Timestamp set ONCE when observer starts (matches customer's pattern) - All events from timestamp forward accumulate (no re-registration) - Remove unused variables: _thresholdSecondsAgo, _bulkEventCount, etc. - Remove unused methods: _clearEvents, _generateBulkEvents, _manualResubscribe - Clean up task-related code (removed _fab, _tasksList, _singleTask) - Remove unused imports (dialog.dart, dql_builder.dart, task.dart) - Add connection info card showing peer connections and transport types - Display connected peers, connection types (Bluetooth, WiFi, WebSocket), and distance - Update app title to "Ditto CPU Reproduction" - Simplify UI to Producer Mode and Consumer Mode sections - Add configurable generation speed (0.5 - 10 events/sec) - Add slider control for dynamic speed adjustment while running - Speed changes update timer interval in real-time - Allows testing callback flooding at different event rates Reproduce customer's SDKS-2280 CPU issue with exact observer pattern: - Fixed timestamp from match start (accumulates ALL events) - No backpressure mechanism (callbacks flood consumer) - Variable speed generation to test callback saturation - CPU spikes (30-40%+) expected with high event rates 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
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.
Summary
Implements fixed timestamp observer pattern with variable speed event generation to reproduce customer's SDKS-2280 CPU usage issue.
Observer Changes
UI Improvements
Event Generator
Purpose
Reproduce customer's exact SDKS-2280 CPU issue:
Testing
Files Changed
lib/main.dart- Fixed timestamp observer, UI improvements, connection infolib/services/event_generator.dart- Variable speed generation🤖 Generated with Claude Code