store function in ref to avoid having to mention it as a dependency

[bacalj]

This stores a ref to getUniqueLocationsInCodapData which needs to be called from a useEffect (and thus the function was supposed to be put in the dependencies). This was causing re-renders and breaking the interaction with CODAP via the API - I think because it was trying to reinitialize each time the component re-rendered.

[pjanik]

Looks good 👍

Given the nature of the problem, I think another solution could be to wrap all these functions in useCallback inside useCodapData. That way, the client wouldn't need to worry about using refs. But I'm speculating.

[kswenson]

Looks good 👍

Given the nature of the problem, I think another solution could be to wrap all these functions in useCallback inside useCodapData. That way, the client wouldn't need to worry about using refs. But I'm speculating.

I agree. A quick glance suggests that some of the functions may not require access to local data, i.e. they could be global functions.