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

remove TauDEM sections from User Guide #1670

Closed
wants to merge 1 commit into from

Conversation

alexbruy
Copy link
Contributor

@alexbruy alexbruy commented Feb 2, 2017

as it is removed from QGIS core and available as 3rd party plugin

Fixes #1619

@DelazJ DelazJ added this to the QGIS 3.0 milestone Feb 2, 2017
@DelazJ DelazJ added the Processing refers to the processing framework settings or features label Feb 2, 2017
@yjacolin
Copy link
Member

yjacolin commented Feb 2, 2017

why remove something still available as third party? Does documentation exist somewhere?

@mach0
Copy link
Member

mach0 commented Feb 2, 2017

Well, documentation exists at the TauDEM Page - and while reading the PR I see that there lots of URLs to old versions included. Is it a common way to include URLs? I tend to only refer to the Homepage so that everyone can find the current version of the software.

@DelazJ
Copy link
Collaborator

DelazJ commented Feb 2, 2017

This is imho one of the questions we will have to answer in the next months with the removal of these providers from Processing but also the move/split of Core plugins (including FTools and Gdal tools) to Processing. What do we do with the current doc and how does it fit the Processing doc structure (if ever kept)?
No opinion, yet...

@alexbruy
Copy link
Contributor Author

alexbruy commented Feb 2, 2017

why remove something still available as third party?

Because it is 3rd party, not core, no? QGIS User Manual contains only QGIS documentation, not documentation for any 3rd party plugin. IMHO it is strange to have docs for tools which are not part of the default installation and should be installed separately.

Does documentation exist somewhere?

Yes, it does. There is a TauDEM site which contains latest documentation. Moreover, our docs were adopted from the original documentation.

@yjacolin
Copy link
Member

yjacolin commented Feb 2, 2017

I agree with @mach0 to add link to home page.

@alexbruy I was just afraid to lose documentation if nothing exist anywhere. So I am +1 with this PR.

@DelazJ
Copy link
Collaborator

DelazJ commented Feb 2, 2017

Because it is 3rd party, not core, no? QGIS User Manual contains only QGIS documentation, not documentation for any 3rd party plugin. IMHO it is strange to have docs for tools which are not part of the default installation and should be installed separately.

Sounds logic, indeed.
Which providers would then be concerned (sorry, I fail to follow which ones were removed and which ones are kept)? But even for kept ones, I think it's more logic to keep documentation at the place where the application is built (ensuring update). So, will they also be removed?
And then, what about translation parts? I mean, browsing quickly the Taudem site, I couldn't find french doc but think it's well translated here. So would we lose that contributions?
There was a discussion begun at http://osgeo-org.1560.x6.nabble.com/New-QGIS-Generic-Help-tt5303312.html. Any news (or should I ask in the ML)?

@DelazJ
Copy link
Collaborator

DelazJ commented Feb 6, 2017

Is there some place left to inform users that there's a possibility to use Taudem in QGIS and if still appropriate, how he could link and make it work within QGIS? Or does a simple installi of Taudem connect it to QGIS Processing?

@pcav
Copy link
Member

pcav commented Feb 6, 2017

It does not. Agreed, it will be important to let users know about TauDEM and OTB availability and functions.

DelazJ referenced this pull request Feb 6, 2017
@alexbruy alexbruy closed this Feb 8, 2017
@alexbruy alexbruy deleted the remove-taudem branch February 8, 2017 10:24
@DelazJ
Copy link
Collaborator

DelazJ commented Feb 8, 2017

@alexbruy Is it because of the discussion in dev ML that you close this PR?

@alexbruy
Copy link
Contributor Author

alexbruy commented Feb 8, 2017

I closed it because there are a lot of concerns about content removal. So better to keep things as it until some general agreement found.

@DelazJ
Copy link
Collaborator

DelazJ commented Feb 8, 2017

imho an open PR gives more opportunity and a place to discuss things than a closed one (and, sometimes better than a ML).
imho again, It's not content removal the main concern. The concern (mine, anyway) is to not remove contents that we could need later or ensure that we point, if needed, the reader to the right place for the information (eg, I found in Victor's commit this information which I guess legitimate the removal of this section : plugin manager is the new provider installer) and because, other than writers, there are also translators behind the scene.

@DelazJ DelazJ added the 3.0 QGIS 3.0 new features label Sep 27, 2019
@DelazJ DelazJ modified the milestones: QGIS 3.0, QGIS 3.4 Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.0 QGIS 3.0 new features Processing refers to the processing framework settings or features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE][processing] remove TauDEM provider from core Processing
5 participants