perf: add useCallback and useMemo to DeveloperEngagementDashboard for… - #99
Open
srajang1805 wants to merge 1 commit into
Open
srajang1805 wants to merge 1 commit into
srajang1805 wants to merge 1 commit into
Conversation
… performance - Added useCallback to loadCSVData function to prevent re-creation on every render - Added useMemo to processedData calculation to prevent reprocessing on every render - Added useCallback to logDashboardState to prevent re-creation of logging function - Updated useEffect dependencies to use the memoized callbacks - Imported useCallback and useMemo from react
|
@srajang1805 is attempting to deploy a commit to the SEETA's projects Team on Vercel. A member of the Team first needs to authorize it. |
Open
58 tasks
This branch has not been deployed
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.
Description
#68
This PR improves dashboard rendering performance by reducing unnecessary function re-creations and repeated data processing during component re-renders.
The primary focus of the changes is optimizing expensive computations and stabilizing callback references to improve rendering efficiency, especially when working with larger CSV datasets or frequently updating dashboard state.
Problem Solved
Previously:
This PR addresses those issues by memoizing expensive calculations and stabilizing function references using React hooks.
Changes
Performance Optimizations
Added
useCallbacktoloadCSVDataAdded
useMemofor processed dataset computationAdded
useCallbacktologDashboardStateUpdated
useEffectdependency arraysCodebase Updates
useCallbackanduseMemofrom ReactTesting / Validation
Screenshots / Demo (if applicable)
N/A — changes are internal performance optimizations with no direct UI modifications.
Additional Notes
Checklist