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

Bump NetTopologySuite from 2.0.0 to 2.1.0 #21932

Merged
2 commits merged into from
Aug 4, 2020

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Aug 4, 2020

Bumps NetTopologySuite from 2.0.0 to 2.1.0.

Release notes

Sourced from NetTopologySuite's releases.

v2.1.0

  • Updated JTS compatibility to 1.17.0.
    • See JTS' own release notes for details of what's included in this; those will not be reproduced here.
    • Note that the API breaking change included in JTS 1.17.0 was not ported to this release. You may safely cast Polygon.ExteriorRing, any element of Polygon.InteriorRings, and the return value of Polygon.GetInteriorRingN to LinearRing instead.
  • Improve xmldoc.
  • Fix some packaging issues.
  • Add GeometryFactoryEx, which allows callers to force polygon rings to be a certain orientation.
  • Unseal NtsGeometryServices in order to allow subclasses to alter the behavior of CreateGeometryFactory.
  • Mark a few useful members on QuadEdge as public instead of internal (they are public in JTS).
  • GML readers read Z values now.
  • CoordinateSequence and Coordinate methods that accept Ordinate enum values now treat values greater than Measure16 the same way they treat all other "Ordinate value is not present in this instance" situations. In the most literal sense, this is technically a behavior breaking change, but only for people who are both casting integer values >32 to Ordinate enum values and actually using them with Coordinate or CoordinateSequence instances with at least 19 dimensions, at least 17 of which are measures.
  • STRtree<T>.Remove now has a possibility of working as expected when T is a value type, instead of never removing anything. It now uses the default equality comparer for the type. Reference types still use object reference equality, for backwards compatibility and performance reasons (it's OK to do this for value types, because Remove was essentially a really slow no-op for them anyway).
  • The internal Coordinate subclass that we use for everything other than XY / XYZ / XYM / XYZM now handles "missing" ordinate values consistently with how other Coordinate types handle them.
  • Copying a Geometry no longer clobbers its GeometryFactory with a brand new one based on the SRID (JTS uses a separate field for this, so this is more faithful).
  • More methods on GeometryFactory have been marked virtual, for consistency with JTS.
  • Fix an incorrect port: GeometryTransformer.Transform was inappropriately transforming a LinearRing into a plain LineString, which is never better.
  • The center of a Bintree Interval is now computed correctly (JTS does not have a method for this; previously, it was the same as the width).
  • Add Envelope.Diameter to get the largest possible distance between any two points that both intersect the bounding box.
  • Fix an issue with VoronoiDiagramBuilder clipping.
  • Start using Microsoft.DotNet.ApiCompat to help us avoid accidentally making API breaking changes.
  • In the same spirit of the improvements to WKT number formatting that came with JTS 1.17.0, also work around a very old bug in .NET Framework / earlier versions of .NET Core before 3.0. For details of the bug and its fix in .NET Core 3.0, see: https://devblogs.microsoft.com/dotnet/floating-point-parsing-and-formatting-improvements-in-net-core-3-0/
  • Add IEntireCoordinateSequenceFilter. Very similar to ICoordinateSequenceFilter, but it is only called once per CoordinateSequence, to enable some significant performance optimizations. We do not currently take advantage of this in NTS, but we're already looking at opportunities, and we're excited to see what you do with it!
Commits
  • 108f840 Update API keys.
  • bb047f8 Merge branch 'develop'
  • c608afd Set JTSCompatibilityLevel to 1.17.0
  • b8e08d2 Fix GML coordinates and srsName parsing
  • 3cc57e2 Expose ordinal constants and use instead of reflection
  • 9a93c73 Convert TestFunction XML files to TestOverlay
  • 626b022 Clarify DistanceOp variable names
  • 247dce2 Add DistanceOp line-line short-circuit optimizations
  • 71649a1 Fix MinimumBoundingCircle.GetMaximumDiameter()
  • ee7e74f Maximum Inscribed Circle and Largest Empty Circle
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually

@dependabot dependabot bot added the auto-merge label Aug 4, 2020
@ghost ghost removed the auto-merge label Aug 4, 2020
@bricelam bricelam self-assigned this Aug 4, 2020
@ghost
Copy link

ghost commented Aug 4, 2020

Hello @bricelam!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 40d99f0 into main Aug 4, 2020
@ghost ghost deleted the dependabot/nuget/NetTopologySuite-2.1.0 branch August 4, 2020 17:54
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants