fixes fetching variable repeated times.#677
Conversation
|
yea, all that ish has gotten so bloated. At some point I will try to a simplification pass on a lot of components. |
There was a problem hiding this comment.
Code Review
This pull request updates the dimension selection and slicing logic to support 4D datasets, including appending the 4D index to the cache key and avoiding slicing off the first dimension in useDataFetcher.tsx. However, removing the 4D slicing logic causes targetDim to incorrectly resolve to the z dimension instead of y for 4D datasets, which needs to be fixed. Additionally, several instances of manual selection initialization in MetaDimSelector.tsx should be simplified using the existing defaultSelection helper, and redundant array scans can be optimized.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
closes #676 #675
We need to clean-up
idx4D. Will keep it for now but it should be removed for all getters, including netcdf ones.