Jump to content
  • 0
Cinquedea

Drop rate exceeding 100%

Question

I was wondering why my drop rate has exceeded 100% in spite of my drop rate which is only 600(x6).

The sticky mucus now has a chance of 120% drop rate. Is there a way to make it max to 100% even if I set the drop rate higher than it is now?

Pls see attached screenies for reference. I'm using rAthena version in Github Thanks in advance.

post-23895-0-69273300-1462378564_thumb.jpg

Link to comment
Share on other sites

8 answers to this question

Recommended Posts

  • 0

No, i left everything untouched exept for the drop rate that i mentioned above.

perhap you should show your settings/changes here.

Link to comment
Share on other sites

  • 0

here's my settings.. I dunno what I did wrong. I hope this helps..

I just set my drop rate to 500%(x5) then an additional VIP drop to 100%.. which sums up to 600%

 

I'm having a feeling that the vip bonus drop overlaps with the current drop rate and doubles the amount which adds additional 20% to the drop chance that makes it 120% instead of 100% max.

 

I also have a similar topic in the source support which has no reply since 4/28/16:

https://rathena.org/board/topic/105723-vip-drop-rate-bonus/

 

 

// Item drop increase. Setting to 0 will disable.

// Note: 50 = 50% item_drop increase.
"// For item_rate = 200: 200 * 50 / 100 = 100 bonus rate added to the 200 base giving total rate of 300."
// Default: 50
vip_drop_increase: 20

post-23895-0-89257500-1462534582_thumb.jpg

post-23895-0-08671100-1462535122_thumb.jpg

Edited by Cinquedea
Link to comment
Share on other sites

  • 0

It's just a display issue, drop rate still is 100%.

 

The drop rate is properly capped to 100% in mob.c::mob_dead.

 

Only the @mi command doesn't cap it in the display, check atcommand.c

#ifdef VIP_ENABLE
			// Display item rate increase for VIP.
			if (pc_isvip(sd) && battle_config.vip_drop_increase)
				droprate += (droprate * battle_config.vip_drop_increase) / 100;
#endif

This code isn't so good because it uses a different formula than what is used in mob_dead. You could just remove it so it no longer is displayed in @mi or copy what is used in mob.c:

			// Increase item drop rate for VIP.
			if (battle_config.vip_drop_increase && (sd && pc_isvip(sd))) {
				drop_rate += (int)(0.5 + (drop_rate * battle_config.vip_drop_increase) / 100);
				drop_rate = min(drop_rate,10000); //cap it to 100%
			}
Link to comment
Share on other sites

  • 0

Replacing the 

#ifdef VIP_ENABLE
			// Display item rate increase for VIP.
			if (pc_isvip(sd) && battle_config.vip_drop_increase)
				droprate += (droprate * battle_config.vip_drop_increase) / 100;
#endif

gives error in compiling... But removing it solves the problem.. But another problem has occured, the drop chance in @mi of non-VIP and VIP players are the same. 

You're a great help, anyway...

Edited by Cinquedea
Link to comment
Share on other sites

  • 0

You can't just replace it with the code in mob.c, you will need to code it yourself to do the same thing.

 

And removing it will still give VIP users more drops, they just won't see it with @mi.

 

Y'know something like:

                drop_rate += (int)(0.5 + (drop_rate * battle_config.vip_drop_increase) / 100);
                drop_rate = min(drop_rate,10000); //cap it to 100%

 

But of course in mobinfo the drop rate variable is called "droprate" and I'm not sure if you can do the rounding the same way, needs to check if it's the same variable type and stuff. Just too lazy to do it now as it's a really simple coding issue.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Answer this question...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use and Privacy Policy.