You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Paging is currently used to parse through granule search results, but it is not the recommended way to interact with the API. The Icesat2Data object and its methods should be modified to not use paging.
The text was updated successfully, but these errors were encountered:
@wallinb - yes! That sounds right, and it would be great if you could make that happen. I'll add you to the organization so that you can create a branch for the work and push changes directly to the repo. By way of orientation: I suspect most of the changes will be in the granules module. We're also encouraging the addition of tests for any new code. Please keep in touch if you have any questions or want a more thorough intro. As always, suggestions and insights are welcome.
@wallinb left NSIDC in 2021 and this issue has languished. I picked up the open PR recently and in the process of updating it learned that the current recommended approach is to use the CMR-Search-After header rather than scrolling (which was preferred over paging). Thus, #87 has been updated to the latest and greatest.
Paging is currently used to parse through granule search results, but it is not the recommended way to interact with the API. The Icesat2Data object and its methods should be modified to not use paging.
The text was updated successfully, but these errors were encountered: