dtarb / TauDEM

Terrain Analysis Using Digital Elevation Models (TauDEM) software for hydrologic terrain analysis and channel network extraction.
http://hydrology.usu.edu/taudem
Other
222 stars 115 forks source link

Distribute extra rows instead of assigning to last rank #244

Open jcphill opened 1 year ago

jcphill commented 1 year ago

Final rank could have up to numranks extra rows, resulting in load and memory imbalance for large parallel runs.

Also remove broken and unused getGridXY() function.

Based on dinf-dxdyc-fix branch due to conflicting redundant change to calling localToGlobal().

jcphill commented 1 year ago

Forgot to mention for regression testing that this pull request will change outputs that are dependent on the parallel decomposition, such as the watershed labels, if the number of processes does not evenly divide the number of rows in the raster. Note that these outputs were inconsistent between runs on different process counts already.