X Tutup
Skip to content

Improvement: Radar auto-zoom on cargo cannisters#5948

Merged
sturnclaw merged 2 commits intopioneerspacesim:masterfrom
mwerle:feat/radar_autozoom
Nov 4, 2024
Merged

Improvement: Radar auto-zoom on cargo cannisters#5948
sturnclaw merged 2 commits intopioneerspacesim:masterfrom
mwerle:feat/radar_autozoom

Conversation

@mwerle
Copy link
Contributor

@mwerle mwerle commented Nov 3, 2024

Similar to auto-zooming on ships, add the capability for the radar to auto-zoom on cargo containers. This is very useful when scooping multiple cargo containers in an area, as otherwise the radar is liable to zoom out to max-zoom-range in-between scooping containers. This makes it very difficult to lock on to the next container.

TODO:

The radar should be switchable between auto and manual modes, with the manual mode being controllable by the keyboard/joystick.

A radar mode and range indicator might also be useful to add.

In future, different radar equipment could provide different ranges and/or capabilities.

Similar to auto-zooming on ships, add the capability for the radar to
auto-zoom on cargo containers. This is very useful when scooping multiple
cargo containers in an area, as otherwise the radar is liable to zoom out
to infinity in-between scooping containers. This makes it very difficult
to lock on to the next container.

TODO: the radar should be switchable between auto and manual modes, with
the manual mode being controllable by the keyboard/joystick. A radar
mode and range indicator might also be useful to add.

In future, different radar equipment could provide different ranges
and/or capabilities.
@bszlrd
Copy link
Contributor

bszlrd commented Nov 3, 2024

The ability to switch radar zoom modes would be most welcome. I think you could do that with the old radar on the old dashboard UI, but not sure if it ever worked.
I'd imagine that there could be a target mode, where your target would define the zoom, and the ship would stay at the edge of the disk until it is close to being in weapon range.

Copy link
Member

@sturnclaw sturnclaw left a comment

Choose a reason for hiding this comment

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

Would you like to track the TODO items somewhere else or address them before merge?

I give this branch my blessing as a useful feature.

@mwerle
Copy link
Contributor Author

mwerle commented Nov 4, 2024

Would you like to track the TODO items somewhere else or address them before merge?

I give this branch my blessing as a useful feature.

I think the TODO is going to be non-trivial (and I mostly wrote it as a braindump rather than something specifically for this PR) but i think the change as-is is already quite valuable and could be merged as an immediate QoL improvement.

Captured the TODO here (please let me know if there's a better place) :

mwerle#3

@sturnclaw
Copy link
Member

That is a good enough spot - do note that you (should) have the ability to push changes to the dev docs now, to document "officially accepted" pipe dreams future development plans.

@sturnclaw sturnclaw merged commit b15d937 into pioneerspacesim:master Nov 4, 2024
@mwerle mwerle deleted the feat/radar_autozoom branch November 4, 2024 23:17
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.

3 participants

X Tutup