-
Notifications
You must be signed in to change notification settings - Fork 268
Lot of improvements in Geometries, Adapters and overall performance #125
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
Conversation
BathoryPeter
commented
Apr 4, 2016
- Z and M support for Geometries and all Adapters (where applicable)
- EMPTY geometry support
- complete centroid calculation, now using de facto standard weighted method
- rewritten WKB and EWKB adapter, supporting bigEndian, Z and M coordinates and SRID
- new TWKB adapter
- new OSM adapter (not complete yet, but has a promising multipolygon converter)
- reduced memory usage
- improved performance in Geometries, eg. built-in geometry operations, array manipulations, constructors, etc.
- improved type detection
- improved and more consistent block-level PHPDoc documentation (not ready)
- more strict OGR standard complaint geometry representation and operations, preserving Postgis compatibility
|
@BathoryPeter Can you delete the unused tags and branches in your fork to speed up composer install? Thanks! :) p/s: You have not enabled Issues on that repo. |
…removed "class" from file names, moved geoPHP.inc to src/geoPHP.php
- Z and M support for Geometries and all Adapters (where applicable) - EMPTY geometry support - completed centroid calculation, now using de facto standard weighted method - rewritten WKB and EWKB adapter, supporting bigEndian, Z and M coordinates and SRID - reduced memory usage - improved performance in Geometries, eg. built-in geometry operations, array manipulations, constructors, etc. - improved type detection - improved and more consistent block-level PHPDoc documentation (not ready) - more strict OGR standard complaint geometry representation and operations, preserving Postgis compatibility
- new OSM adapter (not complete yet, but has a basic multipolygon converter)
…y geometry structures (like geometryReduce()) just builds the most appropriate geometry from the given array of geometries
- fixed a multi-geometry bug in buildGeometry() - changed geoPHP::load() to use buildGeometry as a last step (instead of geometryReduce) - added some more Earth constant
- Add abstract geometry types like Curve, Surface, MultiGeometry - move common methods to abstract classes - add static ::fromArray() method (only to Point and LineString yet) - comment methods and classes - use UnsupportedMethodException::geos() exceptions - rewrite boundary() methods to follow the OGC standard - implemented MultiPoint::isSimple() - lot of minor bug fixes and improvements
…rmula author: Marcus Jaschen, https://github.com/mjaschen/phpgeo
- currently Point and LineString tests are complete (excluding geos fallbacks) - added a phpunit.xml config, creating coverage report - exclude old unit tests from the default config (they are obsolete, not real unit tests, should be rewritten)
(set tolerance in "equals" tests and skip tests where HHVM float precision isn't enough)
| @@ -1 +1,7 @@ | |||
| /vendor | |||
| /tests/coverage/ | |||
| composer.lock | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably include the lock file so that everyone forking and developing would have the same versions of package dependencies?
Only converting malformed xml tags if parsing according to the schema don't bring result
more descriptive exception messages
replace references to phayes repository in readme
…ly if it is malformed
do not create MultiGeometry from geometries if any of them has Data
|
👉 View analysis in DeepCode’s Dashboard |
Composer dropped support of HHVM Travis build fails for PHP 5.5 because archives are only available on trusty
|
Closing this, because my fork got too diverged in the past years. |