Recoil
Recoil copied to clipboard
Optimizing functions 🚀
Memoizing functions using useCallback
and useMemo
for performance benefits :)
Extremely sorry if I made any mistakes :(
@sanjaiyan-dev - Were you able to measure any performance gains with this change?
@drarmstr has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@sanjaiyan-dev has updated the pull request. You must reimport the pull request before landing.
@sanjaiyan-dev has updated the pull request. You must reimport the pull request before landing.
Is there an update on this? We are still seeing that useRecoilValue is quite slow, it is making us consider using a different library
Is there an update on this? We are still seeing that useRecoilValue is quite slow, it is making us consider using a different library
Hi, there are some code changes that need to be done from my side, but I don't know the current status or the maintainer's idea of merging them :(.
This PR is completely broken. Can we just close it please.
This PR is completely broken. Can we just close it please.
Okay :) Should I create a new PR then? (@FezVrasta)
Considering you are mixing up useMemo and useCallback I'm afraid you may not have the required knowledge to contribute to this project at this time. Of course that's just my opinion observing what you wrote so far.
Considering you are mixing up useMemo and useCallback I'm afraid you may not have the required knowledge to contribute to this project at this time. Of course that's just my opinion observing what you wrote so far.
Sorry and you are correct, Can you please clarify if I am correct, useCallback
for memozing functions and useMemo
for return values ? Hope I am correct :)
Hi, anything new about this?