aws-samples / aws-ambit-scenario-designer-ue4

AWS Ambit Scenario Designer for Unreal Engine 4 (Ambit) is a suite of tools to streamline content creation at scale for autonomous vehicle and robotics simulation applications.
https://aws-samples.github.io/aws-ambit-scenario-designer-ue4/
Apache License 2.0
85 stars 21 forks source link

Update screenshots in user guide with latest UI #12

Closed rukekre closed 2 years ago

rukekre commented 2 years ago

What was the problem/requirement? (What/Why)

The user doc contained certain UI images that were older and didn't include newer functionality.

What was the solution? (How)

Relevant screenshots have been updated in the user guide to match the latest UI. Changes have been made to the text to include any new information.

What artifacts are related to this change?

Issues: (internal) P54511997

What is the impact of this change?

Any users referring to the user guide will be provided with the latest information.

Are you adding any new dependencies to the system?

No.

How were these changes tested?

Locally, using the standard procedure to view changes to the user doc.


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

kris-work commented 2 years ago

Nit: images/spawn-with-houdini/AmbitSpawnerForHoudini_UI_Annotated.png doesn't fully match the style of other image annotations. The wrong font is used, font size relative to circle size is off, lines have rounded ends instead of square.

kris-work commented 2 years ago

Nit: /images/spawn-vehicle-path/AmbitVehiclePathGenerator_UI.png: Annotations have visual artifacts that the other images don't. Specifically, aliasing of the circles. Also, the numbers in the circles are the wrong weight and aren't all centered well.

kris-work commented 2 years ago

Images in individual-scenario-generation.md:

kris-work commented 2 years ago

Images in bulk-senario-generation.md:

Images in individual-scenario-generation.md:

kris-work commented 2 years ago

The file bulk-senario-generation.md has a typo in its name. It should be bulk-scenario-generation.md. I haven't marked this as a "nit" since it is customer-facing (appears in the URL of the generated docs).