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

2024 field image json - appears to still be incorrect #6194

Closed
jsimpso81 opened this issue Jan 10, 2024 · 2 comments · Fixed by #6185
Closed

2024 field image json - appears to still be incorrect #6194

jsimpso81 opened this issue Jan 10, 2024 · 2 comments · Fixed by #6185

Comments

@jsimpso81
Copy link

Describe the bug
After the update the JSON file for the 2024 field image appears to still be incorrect.

The value 1008 probbly should be 1108.

Also curious how the values were derived?

In looking at a zoomed in file, it seems that values should be 112, 27, 1108, 522. Except for the 1008, these are only very slightly different than the JSON.

To Reproduce
Steps to reproduce the behavior:

  1. Use the field in one of the applications that displays fields and visually review.

Expected behavior
The entire field should be viewable.

Screenshots
N/A

Desktop (please complete the following information):

  • Windows 10 (Used Paint to view zoomed in image..)
  • N/A

Additional context
Add any other context about the problem here.

@PeterJohnson PeterJohnson linked a pull request Jan 10, 2024 that will close this issue
@sciencewhiz
Copy link
Contributor

Yes the 1008 is a typo, #6185 changes to a better image and fixes the dimensions. I suspect that the +/- 2 pixel is just a difference in exactly where it's measured, since there's a gradient.

@shueja
Copy link
Contributor

shueja commented Jan 12, 2024

Also curious how the values were derived?

Typically, one uses the ancient method of looking at the pixel coordinates of the cursor in MS Paint to find the coordinates of each wall.

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 a pull request may close this issue.

3 participants