Skip to content

Conversation

@rouault
Copy link
Member

@rouault rouault commented Feb 10, 2026

No description provided.

@rouault rouault added this to the 3.13.0 milestone Feb 10, 2026
@rouault rouault added the funded through GSP Work funded through the GDAL Sponsorship Program label Feb 10, 2026
@rouault rouault force-pushed the GDALGetNumThreads branch 4 times, most recently from 3971ac6 to 8ddf9f0 Compare February 11, 2026 02:38
@coveralls
Copy link
Collaborator

coveralls commented Feb 11, 2026

Coverage Status

coverage: 71.687% (-0.003%) from 71.69%
when pulling a5fcccd on rouault:GDALGetNumThreads
into 49816e2 on OSGeo:master.

Copy link
Collaborator

@elpaso elpaso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see there are different default values for nMaxThreads can you add some comments to explain the different choices?

If is there a (most common) default perhaps we could make it a global named constant.

@rouault
Copy link
Member Author

rouault commented Feb 12, 2026

we could make it a global named constant.

good suggestion. GDAL_DEFAULT_MAX_THREAD_COUNT added and used wherever we arbitrarily used 128 or 1024

@rouault rouault merged commit 489b08b into OSGeo:master Feb 12, 2026
3 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

funded through GSP Work funded through the GDAL Sponsorship Program

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants