Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The code in this repository (https://github.com/dtarb/TauDEM) does not (yet) have the improvements that Kornelijus implemented and reported in the XSEDE paper. Kornelijus' code is in https://github.com/kornholi/TauDEM. #268

Open
mperjak opened this issue Nov 15, 2024 · 2 comments

Comments

@mperjak
Copy link

mperjak commented Nov 15, 2024

          The code in this repository (https://github.com/dtarb/TauDEM) does not (yet) have the improvements that Kornelijus implemented and reported in the XSEDE paper.  Kornelijus' code is in https://github.com/kornholi/TauDEM.

It is my intention to merge the two forks together, but there were some result differences that I wanted to track down before merging, and also a different way of handling partition memory that required additional dependencies that needed some sorting out. These details and lack of time have led to this not being merged yet.

Originally posted by @dtarb in #115 (comment)

@mperjak
Copy link
Author

mperjak commented Nov 15, 2024

Hello! Was this code ever merged into one of your branches? We are trying to use Amazon's Parallel Cluster to parallelize our process, but have been stuck on pitremove for a couple of months now.

@jcphill
Copy link

jcphill commented Nov 15, 2024

You could also try https://github.com/jcphill/TauDEM/tree/all-2022 which is not 2-D but has other improvements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants