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

Promoting ogr_layer_algebra to official python script #6483

Closed
wants to merge 4 commits into from

Conversation

kssvrk
Copy link
Contributor

@kssvrk kssvrk commented Oct 7, 2022

What does this PR do?

This PR Promotes ogr_layer_algebra.py to official Python script mentioned at issue 1581.

What i have done is the following.

  • move ogr_layer_algebra.py from swig/python/gdal-utils/osgeo_utils/samples to swig/python/gdal-utils/osgeo_utils/
  • create a calling gdal-utils script for ogr_layer_algebra.py in swig/python/gdal-utils/scripts
  • Change the arguments of main function of ogr_layer_algebra from argv=None to argv=sys.argv
  • created documentation for ogr_layer_algebra at doc/source/programs/
  • Linked the documentation for ogr_layer_algebra in index under vector section
  • Created test cases for all modes of ogr_layer_algebra at autotest/pyscripts/test_ogr_layer_algebra.py

What are related issues/pull requests?

Tasklist

  • Add test case(s)
  • Add documentation
  • Updated Python API documentation (swig/include/python/docs/)
  • Review
  • Adjust for comments
  • All CI builds and checks have passed

@kssvrk kssvrk changed the title Ogr layer algebra Promoting ogr_layer_algebra to official python script Oct 7, 2022
@kssvrk
Copy link
Contributor Author

kssvrk commented Oct 7, 2022

This pull request has non standard documentation , i am sorry for the PR.
Just deployed the documentation and checked. Will submit another PR shortly. Closing this request.

First time contributor for gdal :)

@kssvrk kssvrk closed this Oct 7, 2022
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

Successfully merging this pull request may close these issues.

None yet

1 participant