Skip to content

adds a one-way fly_to_command traitlet enabling flyTo from JS#1201

Closed
batpad wants to merge 1 commit into
mainfrom
feature/fly-to-trait
Closed

adds a one-way fly_to_command traitlet enabling flyTo from JS#1201
batpad wants to merge 1 commit into
mainfrom
feature/fly-to-trait

Conversation

@batpad

@batpad batpad commented Jun 7, 2026

Copy link
Copy Markdown
Member

Currently, if one wants the map to flyTo a location, one has to send a message through Jupyter comms. Thus, it is not possible to update the map view when lonboard is rendered in static contexts.

@kylebarron I know this is something you're not keen on supporting :( and I can see that the design is a lot nicer when going through comms. But it would be really nice to have some way to update the viewport in a frontend-only environment.

This adds a fly_to_command traitlet that is attached to the shared model, so can be set in frontend JS. This implementation should work for me, but am not sure of the naming, etc. Just putting this up as a draft PR and then we can discuss.

@kylebarron

Copy link
Copy Markdown
Member

Ok so if I understand:

From a client side only app you're not able to go through Jupyter Comms? Are you able to mock it some way so that the Lonboard JS side doesn't have to change?

The bigger question I think is that if we proceed on this sort of client-side work, do we need a client-side public API separate from the Python API?

@batpad

batpad commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

@kylebarron your idea of just mocking the comms on the frontend when a kernel isn't present seems like it should work, so we don't need to have these two divergent paths in lonboard.

Closing this, pretty sure the work-around is fine, and maybe in fact useful as a generic way to mock comms communication for static widgets. Thanks!

@batpad batpad closed this Jun 9, 2026
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