[POC] 558 delete richdem from source #612
Open
+259
−304
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR resolves the ticket #558 that requires the removal of the
RichDEM
dependency from the core functionnality of thexDEM
project to ensure the project's license is not impacted byRichDEM
's GNU GPL v3.0 license.The changes made in this PR remove
RichDEM
from operational code and ensure thatRichDEM
is only used for testing purposes. Additionally, documentation and contribution guides have been updated to reflect this adjustment.Changes Implemented
Dependency Management
Updated
setup.cfg
anddev-environment.yml
to move therichdem
package from the mainopt
dependencies to thetest
section.Source modifications
terrain.py
anddem.py
, theuse_richdem
parameter was removed from functions.RichDEM
were completely removed.Tests adjustments
RichDEM
is still used in tests to ensure that all attribute functions give the same results as those ofRichDEM
. The corresponding part of the code that has been removed from source was transfered toconftest.py
.Documentation Updates
use_richdem
parameter from functionsRichDEM
is only used for testing purposes and is not required for the operation ofxdem
.RichDEM
should only be used in tests, and core functionality ofxdem
must not rely onRichDEM
to comply with the license.