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

Add new QEP-0-Template.md #326

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add new QEP-0-Template.md #326

wants to merge 1 commit into from

Conversation

DMZHY
Copy link

@DMZHY DMZHY commented Feb 11, 2025

Add QEP-0-Template.md and its images.

@troopa81
Copy link

Could you change the QEP name so it state explicitly what this QEP is about.

@troopa81
Copy link

Thank you for your proposal @DMZHY and your interest in QGIS.

I have to admit I'm not a great fan about integrating more and more proprietary database in QGIS core. We already have this discussion with SAP Hana support and we accepted to integrate it into core. There is also already Oracle and MSSQL. So why not Dameng?

Did you consider writing a python plugin? We did it for QDuckDb (which is OpenSource). It works well (There is, I have to admit, some performance implication that might require optimization on core side).

@rouault
Copy link
Contributor

rouault commented Feb 11, 2025

I have to admit I'm not a great fan about integrating more and more proprietary database in QGIS core

Speaking only for myself:

I believe one of our (= QGIS community) concern is that those type of contributions tend to be extremely focused to the provider that the company contributes to (in the best case, they maintain it. in the worse cases, they end up abandoning it, like the DB2 provider we ended up removing), and not participate actively to the rest of the general life of the project. So the project ends up with big chunks of code that are only of use for a very small subset of its users (or at least the part of the users the active contributors usually interact with. I'm well aware that China is a big country and the potential number of users of this database could be big), but that become a liability for the rest of the contributors that have to deal with them on a daily basis and do lots of micro-maintenance tasks on them each time a small or big evolutions of the provider API is done.

My recommendation would be that before pushing a big "present" to QGIS you start by finding your place in the contributing community by helping addressing known issues, small new features of general interest, participating in code reviews, etc. That is showing you care about QGIS by itself, and not just about increasing the market share of your product. That's an on-boarding process that usually takes months.

@nyalldawson
Copy link
Contributor

nyalldawson commented Feb 13, 2025

@DMZHY

A couple of notes:

  • you need to copy the QEP-0 template file and then modify the copy, not replace the existing content. Add a descriptive title for this pull request, and put SOMETHING in the pull request description. Don't be sloppy here -- you're supposed to be showcasing your willingness to work hard and implement quality code and fit in with QGIS development practices, not just do the bare minimum and require others to work to interpret your meaning 🤔
  • The proposal is missing technical details -- in its current form all your saying is basically "the same as postgres, but for Dameng". Instead, you should focus on what the technical consequences of this proposal are. Does it add extra dependencies to QGIS? Will it always be compiled, or conditionally based on other libraries? How well packaged are those libraries? What CI setup will you implement to test the provider? Which parts of the Dameng spatial database DON'T fit within the QGIS vector data provider model and what trade-offs do you need to make?

And then, as @rouault has pointed out, you need to also describe your plans for maintaining the provider. Who is responsible for bug fixing? Who will maintain the CI setup? Will you be financially sponsoring the QGIS project to help offset the added maintenance burden of adding a new provider?

@haubourg
Copy link
Member

Agreed with all this.
Please really use QEP title and body to summarize what is detailed in the files.
This will make it a lot easier to review for us all.

@nyalldawson nyalldawson added Project A proposal which concerns a project, eg new functionality In Discussion QEPs currently in discussion stage labels Feb 13, 2025
@DMZHY
Copy link
Author

DMZHY commented Feb 14, 2025

Dear QGIS Development Team,

First of all, thank you for your attention and feedback on our enhancement proposal for QGIS adaptation to Dameng Database.

I understand your concerns about integrating more proprietary databases into the QGIS core code, and I fully agree that as QGIS core functionality continues to expand, greater emphasis should be placed on maintaining the universality and sustainability of the code. Our company places great emphasis on the development of spatial databases, and currently, Dameng Database has a dedicated R&D team focused on spatial data. As QGIS becomes widely adopted in China, many QGIS users in the market have requested support for Dameng Database. Therefore, our R&D team has undertaken adaptation work for QGIS. Regarding the code submission for QGIS, we commit to providing ongoing maintenance and support to ensure maintenance and optimization of the adaptation code in the long run.

In addition to database adaptation, our R&D team contributes to various aspects of QGIS open source. We will actively participate in issue reporting, bug fixes, and addressing community issues within the QGIS open-source community, striving to contribute to the development of QGIS. At the same time, we have completed corresponding adaptation work on other GIS platforms such as ArcGIS and SuperMap. Our company has also provided feedback on implementation issues in GIS open-source communities such as GEOS, CGAL, and GDAL.

We hope to have the opportunity to contribute our adaptation code to the QGIS core code. Of course, if it currently does not meet the integration requirements for certain reasons, we fully understand and support that decision. Once this proposal is accepted into the core code for QGIS, we will be sure to modify the copy file name in the format of "QEP-0-QGIS-data-provider-for-Dameng.md". We will also include details on the technical ramifications brought by this proposal, such as extra dependencies to QGIS and potential tradeoffs for using Dameng.

In the meantime, we are very interested in the Python plugin approach. May we ask if it would be possible to consider acceptance and integration through a Python plugin?

Once again, thank you for your invaluable feedback and suggestions. We look forward to an extended collaboration with the QGIS community to contribute to the development of QGIS.

@troopa81
Copy link

In the meantime, we are very interested in the Python plugin approach. May we ask if it would be possible to consider acceptance and integration through a Python plugin?

Any python plugins can be uploaded in the plugin platform and could be then installed by any user using the extension widget.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In Discussion QEPs currently in discussion stage Project A proposal which concerns a project, eg new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants