Skip to content

Conversation

@seisman
Copy link
Member

@seisman seisman commented Jan 7, 2026

Recently, the docs building has been failing frequently (xref: https://github.com/GenericMappingTools/pygmt/actions/workflows/ci_docs.yml):

    Unexpected failing examples (1):
    
        ../examples/tutorials/advanced/draping_on_3d_surface.py failed leaving traceback:
    
        Traceback (most recent call last):
          File "/home/runner/work/pygmt/pygmt/examples/tutorials/advanced/draping_on_3d_surface.py", line 102, in <module>
            with rasterio.open(url_to_image) as dataset:
                 ~~~~~~~~~~~~~^^^^^^^^^^^^^^
          File "/home/runner/micromamba/envs/pygmt/lib/python3.14/site-packages/rasterio/env.py", line 464, in wrapper
            return f(*args, **kwds)
          File "/home/runner/micromamba/envs/pygmt/lib/python3.14/site-packages/rasterio/__init__.py", line 367, in open
            dataset = DatasetReader(path, driver=driver, sharing=sharing, thread_safe=thread_safe, **kwargs)
          File "rasterio/_base.pyx", line 329, in rasterio._base.DatasetBase.__init__
        rasterio.errors.RasterioIOError: HTTP response code: 429

It seems the 1000ppx image used by this example no longer exists, and is redirected to another resolution, which causes the 429 (Too many requests) error. This PR replaces it with the 1024px image instead.

@seisman seisman added the documentation Improvements or additions to documentation label Jan 7, 2026
@seisman seisman added this to the 0.18.0 milestone Jan 7, 2026
@seisman seisman added skip-changelog Skip adding Pull Request to changelog final review call This PR requires final review and approval from a second reviewer labels Jan 7, 2026
@seisman seisman merged commit b243fef into main Jan 7, 2026
13 of 15 checks passed
@seisman seisman deleted the fix/draping branch January 7, 2026 06:21
@seisman seisman removed the final review call This PR requires final review and approval from a second reviewer label Jan 7, 2026
@weiji14
Copy link
Member

weiji14 commented Jan 8, 2026

Seems like the new image is also producing a 429 error? Should we just copy that PNG file and put it on the GMT server cache, similar to https://github.com/GenericMappingTools/gmtserver-admin/blob/master/cache/euflag.nc?

@seisman
Copy link
Member Author

seisman commented Jan 8, 2026

Sounds good.

@seisman
Copy link
Member Author

seisman commented Jan 8, 2026

Seems like the new image is also producing a 429 error?

It appears the 429 error occurs less frequently with the new image.

@yvonnefroehlich
Copy link
Member

yvonnefroehlich commented Jan 8, 2026

Seems like the new image is also producing a 429 error?

It appears the 429 error occurs less frequently with the new image.

For me, running this tutorial works for the 1024px image, but fails for the 1000px image ("RasterioIOError: HTTP response code: 429").

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants