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

[Crash][1.21.1][Fabric]: Crash upon opening waypoint manager #780

Closed
DisdainofDeimos opened this issue Aug 30, 2024 · 10 comments
Closed

[Crash][1.21.1][Fabric]: Crash upon opening waypoint manager #780

DisdainofDeimos opened this issue Aug 30, 2024 · 10 comments

Comments

@DisdainofDeimos
Copy link

What happened?

After setting up waypoint groups and color schemes (using forced color override feature for grouping), the game now crashes upon opening the waypoint manager screen. This is replicate-able from both the waypoint manager hotkey, and the waypoint manager button in the fullscreen map. Crash report with mod list link

Seems to be a problem with the getColor being = null, I am unsure as to the reason for this.

Mod Loader

Fabric

Minecraft Version

1.21.x

Version

6.0.0

Relevant log output

crash-2024-08-30_14.46.22-client.txt
journeymap.log
latest.log

@mysticdrew
Copy link
Member

aaah okay, likely because you didn't set a color when you saved a force override. I will get this fixed, thanks for the report.
While testing this, looks like there is an issue with changing colors of a group too for some reason, it worked at one point. I guess I broke it somehow.

@mysticdrew
Copy link
Member

if you want to pass me your waypointdata.dat file in here I can fix it quick for you so you can use your waypoint manager while I work on releasing a fix.
it is located in C:\Users\matta\AppData\Roaming\ModrinthApp\profiles\2024 - Alibrealm (3)\journeymap\data\mp\Squeak\waypoints\WaypointData.dat
Just drop it in a response here.

@DisdainofDeimos
Copy link
Author

Yeah I did some further testing just now, and the waypoint manager still works just fine in a fresh singleplayer world without any waypoints or groups, so your explanation seems to fit! Thanks for the quick response, here's that WaypointData file (it won't let me upload a .dat so I changed the file extension to .txt temporarily, I can also put a drive link if changing the file extension messes something up!)
WaypointData.dat.txt

Thanks again <3

@mysticdrew
Copy link
Member

WaypointData.dat.txt
See if this works for you.

@DisdainofDeimos
Copy link
Author

image
Were back in business!
Thanks so much for your time and for the personal fix <33
Love your work, thanks for everything you give the community!

@mysticdrew
Copy link
Member

image Were back in business! Thanks so much for your time and for the personal fix <33 Love your work, thanks for everything you give the community!

Awesome thanks! I will get this bug fix out tomorrow hopefully.
Other than this issue, what are your thoughts on the new waypoint manager?
Like it, love it, hate it? Improvements I can make?

@DisdainofDeimos
Copy link
Author

I love it! It's great being able to have groupings although I would like to see a quick and easy button to instantly hide/show whole groups of waypoints at a time, and maybe expanding on the icon choices for both individual waypoints and groups? All-in-all you've made (IMO) the best Minecraft mapping mod for both casual and veteran/technical players, great stuff!

@mysticdrew
Copy link
Member

I love it! It's great being able to have groupings although I would like to see a quick and easy button to instantly hide/show whole groups of waypoints at a time, and maybe expanding on the icon choices for both individual waypoints and groups? All-in-all you've made (IMO) the best Minecraft mapping mod for both casual and veteran/technical players, great stuff!

Icons are coming for groups.
So more icons can be added easily via resource packs. I only provided the default ones for now
Details on adding new icons via resource packs. (not documented yet)
TeamJM/journeymap-docs#90

The group panel is getting a bit squished already, we are a bit limited on space, I cannot make it larger really because unlike you and I, many people still do not adjust their GUI scale. So I have to make it all fit in scale 4, which sucks. but I will see what I can do.

@mysticdrew
Copy link
Member

image
I was able to make it work :)

@DisdainofDeimos
Copy link
Author

Awesome! That will be a really convenient feature to have especially on large / long playtime servers/worlds with lots of waypoints! Great stuff friend :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants