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

[0.49.15] (Minor Visual Bug) Sensor Range Bands Not Showing Until Unit Movement Begins #4795

Closed
PhoenixHeart512 opened this issue Sep 18, 2023 · 6 comments · Fixed by #4946
Closed
Assignees
Labels

Comments

@PhoenixHeart512
Copy link

PhoenixHeart512 commented Sep 18, 2023

Fill out the fields below with as much pertinent information as possible to help the developers solve the issue.
Search for your issue inside open and closed issues first before reporting so as not to make duplicates.
(Delete the above lines)

Environment

megamek.MegaMek.initializeLogging(MegaMek.java:118) - Starting MegaMek v0.49.15-SNAPSHOT
Build Date: 2023-09-18T01:07:34.461102200
Today: 2023-09-18
Origin Project: MekHQ
Java Vendor: Eclipse Adoptium
Java Version: 17.0.6
Platform: Windows 10 10.0 (amd64)
System Locale: en_US
Total memory available to MegaMek: 8 GB

Description

During movement phase, the light blue sensor range bands are not showing up until I actually make a move with a unit (even if it's just changing facing).

Included are my MM preferences and gameoptions, I have double blind/TacOps Sensor Rules both enabled. Issue occurs while using the ctrl-Q movement envelope, as well as when not using ctrl-Q.

Once I move a unit to get them to show up, if I then click to a different unit and back the sensor bands disappear until the unit is moved again.

They are clearly functional, as I am able to detect units with sensors even when they're not in LOS, this is purely a visual indication of the detection range of my units.

Files

Sensor Bands Movement Phase.zip

Prior to moving the unit:
Sensor_Bands_Missing

After changing facing 1 hexside to the left:
Sensor_Bands_Showing

(edited to remove blank space on screenshot and re-upload zip)

@Sleet01
Copy link
Collaborator

Sleet01 commented Sep 26, 2023

@PhoenixHeart512 Would it be possible to provide an MM version number where this works as expected, for comparison? I tried 0.48.0 but it doesn't appear to have this functionality.

There has been some churn in the code that handles drawing sensor ranges within the last couple quarters, so I'd like to take a look at a version that works correctly and compare code.

@Sleet01 Sleet01 self-assigned this Sep 26, 2023
@PhoenixHeart512
Copy link
Author

I still have all my old builds, I'll see if I can dig through and find one. Do you remember which version specifically they were added in? Ideally which nightly build date, since I have a bunch

@Sleet01
Copy link
Collaborator

Sleet01 commented Sep 26, 2023

@PhoenixHeart512 I was just hoping to get a version number from you, like "0.49.9" or something, from the last time that you saw the sensor overlay working correctly.

It appears that a bunch of changes went in around May of 2023; if you have something from April/May of this year that should work. If not, I can trawl through the commit messages and try to pinpoint where stuff changed.

@PhoenixHeart512
Copy link
Author

It looks like May 30th is when I noticed the sensor bands change, so ill check that build when I get home

@PhoenixHeart512
Copy link
Author

I should have that build still, I'll zip it up and send it to you when I get home from work

@HoneySkull
Copy link
Collaborator

I provided a fix for this bug according to how it should work.

  1. When an entity is selected and sensor ranges is enabled, the ranges should be rendered for the selected entity and be updated when a new entity is selected for that selected entity.
  2. Toggling sensor ranges should show and hide sensor ranges. (There was bug that if you turned off sensor ranges then selected a different entity and then enabled sensor ranges, the ranges would not display until some GUI event triggered the re-calculating of ranges.)

SJuliez added a commit that referenced this issue Dec 12, 2023
Fix #4795 Sensor Range Bands Not Showing Until Movement Begins.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants