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 support for standalone Places, Routes SDKs #40

Merged
merged 34 commits into from
Oct 30, 2024
Merged

Conversation

mbalfour-amzn
Copy link
Contributor

Description of changes:

This adds support for converting all of the new response types in the standalone Places and Routes SDKs. See the README for more details on the new methods and how to use them.

dependabot bot and others added 30 commits September 11, 2024 17:02
Bumps [micromatch](https://github.com/micromatch/micromatch) from 4.0.5 to 4.0.8.
- [Release notes](https://github.com/micromatch/micromatch/releases)
- [Changelog](https://github.com/micromatch/micromatch/blob/master/CHANGELOG.md)
- [Commits](micromatch/micromatch@4.0.5...4.0.8)

---
updated-dependencies:
- dependency-name: micromatch
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
…romatch-4.0.8

Bump micromatch from 4.0.5 to 4.0.8
Bumps [fast-xml-parser](https://github.com/NaturalIntelligence/fast-xml-parser) to 4.4.1 and updates ancestor dependency [@aws-sdk/client-location](https://github.com/aws/aws-sdk-js-v3/tree/HEAD/clients/client-location). These dependencies need to be updated together.


Updates `fast-xml-parser` from 4.2.5 to 4.4.1
- [Release notes](https://github.com/NaturalIntelligence/fast-xml-parser/releases)
- [Changelog](https://github.com/NaturalIntelligence/fast-xml-parser/blob/master/CHANGELOG.md)
- [Commits](NaturalIntelligence/fast-xml-parser@v4.2.5...v4.4.1)

Updates `@aws-sdk/client-location` from 3.359.0 to 3.649.0
- [Release notes](https://github.com/aws/aws-sdk-js-v3/releases)
- [Changelog](https://github.com/aws/aws-sdk-js-v3/blob/main/clients/client-location/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-js-v3/commits/v3.649.0/clients/client-location)

---
updated-dependencies:
- dependency-name: fast-xml-parser
  dependency-type: indirect
- dependency-name: "@aws-sdk/client-location"
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
…ti-d875a7cb14

Bump fast-xml-parser and @aws-sdk/client-location
On further investigation, Polygon should be fully flattened, and none
of the other fields listed actually appear in supported Response types.
For the new GeoPlaces APIs, each Response type is now mapped
to a separate API. This makes the code easier to write and maintain,
and will scale better if more APIs are added over time.
Add places V2 API support to datatypes lib
Bumps [rollup](https://github.com/rollup/rollup) from 3.22.0 to 3.29.5.
- [Release notes](https://github.com/rollup/rollup/releases)
- [Changelog](https://github.com/rollup/rollup/blob/master/CHANGELOG.md)
- [Commits](rollup/rollup@v3.22.0...v3.29.5)

---
updated-dependencies:
- dependency-name: rollup
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
MapLibre expects GeoJSON Feature ids to be integers.
By placing PlaceId values in the id field, they get stripped by
MapLibre into "undefined". However, because PlaceId is a valid
Feature id value for uses other MapLibre, this change changes the
place-converter to stop stripping PlaceId from properties and leaves
the id alone for backwards compatibility. The new geoplaces-converter
has been changed to put a unique integer in the id field and to stop
stripping PlaceId from properties.
…lup-3.29.5

Bump rollup from 3.22.0 to 3.29.5
Fixed Places Feature id handling to be more MapLibre-friendly
This converter is more complex than the V0 converter. It exposes
options for converting leg lines, travel step lines, travel step
points, leg arrival/departure points, and span lines.
When all of these are combined in a single GeoJSON, they can be
drawn, detected, and styled differently based on the FeatureType
property added to each generated Feature.
This API change replaces Query with QueryText.
Adds route converter for V2 CalculateRoutesResponse
Fixed some README method name typos
* Added V2 route converters and changed flattenProperties default to true.
Added converters for CalculateIsolines, OptimizeWaypoints, and
SnapToRoads. Also changed all of the flattenProperties defaults for the
V2 converters to true because that's the common use case path for use
with MapLibre.

* Updated all comments to include examples of rendering the results.
Also made everything more typedoc-friendly.

* Fixed up README and comments.

* Removed unneeded comments.

* Addressed PR feedback.

* Renamed feature types based on PR feedback.

* Optimized and split the extract methods.

* Fixed README to match PR feedback changes.
The UMD build should be used for browsers (and unpkg, which otherwise
references `main`, which is now CommonJS).
Specify browser artifacts in package.json
The package names changed (now geo-<vertical>) and a property was
removed from Routes.
* Switched to public polyline release.

* Switch to released polyline package.
Previous commit was relying on a locally-installed
development build of @aws-geospatial/polyline, which is why
it didn't cause any compile failures.

* Added commonjs to the datatypes rollup
Also added a CHANGELOG.md file.
Bumps [rollup](https://github.com/rollup/rollup) from 3.22.0 to 3.29.5.
- [Release notes](https://github.com/rollup/rollup/releases)
- [Changelog](https://github.com/rollup/rollup/blob/master/CHANGELOG.md)
- [Commits](rollup/rollup@v3.22.0...v3.29.5)

---
updated-dependencies:
- dependency-name: rollup
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
@mbalfour-amzn mbalfour-amzn requested a review from a team as a code owner October 30, 2024 22:30
@mbalfour-amzn mbalfour-amzn merged commit ae072c8 into main Oct 30, 2024
1 check passed
@mbalfour-amzn mbalfour-amzn deleted the staging_changes branch October 30, 2024 22:39
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.

5 participants