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 Isochrone tool #97

Merged
merged 1 commit into from
Aug 19, 2021
Merged

Conversation

frasercl
Copy link

Adds a new Isochrone tool for analyzing and planning traverses. Features:

  • Render isochrones as raster layers showing color-ramped accumulated cost out to a set maximum
  • Hover over isochrones to show least-cost paths and minimum cost from start point to hovered point
  • Separated cost model interface allows easy extension and switching models on-the-fly
  • Add, remove, and separately edit multiple isochrones
  • Configurable color ramps, color steps, opacity, resolution, data sources

* Moves from a pixel to its 8 immediate neighbors.
* A purely distance-based model will produce an octagon with these moves.
*/
const queenMoves = [
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So she's restricting her movement to not make the king feel inferior?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't know a lot about chess haha - I believe that's a term I saw somewhere else, and I didn't think about it too much. Would probably more correctly be called kingMoves.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

queen is fine; the comment and 8 elements explain it pretty easily.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And those moves can compose to queen moves, so it's not as if it's completely inaccurate.

Copy link
Member

@tariqksoliman tariqksoliman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and thanks for pulling this in!

@tariqksoliman tariqksoliman merged commit ba6f836 into NASA-AMMOS:development Aug 19, 2021
@frasercl frasercl deleted the isochrone branch August 20, 2021 15:43
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.

2 participants