Incorrect Crown Sizes

Question

Incorrect Crown Sizes

Closed this issue 4 years ago · 9 comments

jet082 commented 4 years ago

The crown size for Scarred Yian Garuga is incorrect. The crown size 1708.48 is indeed gold, but it shows up as silver.

I believe that this is a rounding error and I believe it also impacts many more crown possibilities than this one.

Answer 1 · 2020-03-31T19:25:15.000Z

The Blackveil mini crown (1885.86) also does not display correctly.

I believe the regular Yian Garuga crowns also have issues.

Answer 2 · 2020-04-01T03:45:22.000Z

This is a rounding issue, and I recall it was reported before: gabrielefilipp#45

I'll try and take a look at it later this week.

Answer 3 · 2020-04-03T22:04:47.000Z

looking at this a little more I'm not sure where the rounding issue comes into play. I feel like it only impacts crowns on the edge of the threshold.

The monster's ModifiedSizeScale is calculated from it's SizeScale/Scale Modifier (pulled from memory). If I happen to see a monster close to the boundary or if someone knows a specific quest/event that hits the right conditions I'll look into it, but for now I'm going to leave it alone.

Answer 4 · 2020-04-04T00:12:51.000Z

I managed to find one monster at the limit and it looked good to me.

I'll see if I can find it on a Yian or Blackveil, but I think this is working on the latest version.

Answer 5 · 2020-04-04T01:42:50.000Z

found a silver 1625.14 S. Yian. and it was labeled correctly.

I'll keep an eye out, but I think this is fixed in the latest version. I have some code clean up/more safi clean up, so I'll cut another version this weekend.

Answer 6 · 2020-04-04T16:10:59.000Z

Well, a lot of the crowns "on the edge" do work from my experience. However, a handful do not.

If you want to duplicate these conditions, you can use this mod (specifically the Guaranteed Big Crowns one) and then enter the Scars Tell the Whole Story Event Quest.

The crown should show up as gold, but it will show up as silver on SmartHunter.

Answer 7 · 2020-04-04T16:16:36.000Z

Thanks for the link, I'll give it a go this afternoon.

Answer 8 · 2020-04-04T21:10:13.000Z

Yian Garuga : 1708.482
SS:     0.983999968
SM:     0.8
MSS:    1.2299999

Found where I can add a round to fix it.

This should fix all the edge cases.

I'll make a release later today.

Answer 9 · 2020-04-05T12:49:26.000Z

It might be worth testing Blackveil as well. You can test that with the same mod (but the mini crown version) and the 5* Here Comes the Deathmaker quest.

Under those conditions it should show up as 1885.86 and should show a mini crown.

That way you can test both the mini and the giant edge cases.