Skip to content
This repository has been archived by the owner on Apr 29, 2020. It is now read-only.

Fishing skill up #6100

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

nuno-pires-cruz
Copy link
Contributor

Implementation of Fishing skill up logic.

Chance for skill up formula, similar to craft, but with a twist. The higher the difference between user skill and fish max skill needed, lower the chance to skill up.

Uses same craft vars, from map config file, for control of skill up chance and skill up multiplier.

Added new var to map config file, fishing_daily_limit, to enable a daily limit, like on retail.

Any input is welcome! Feel free to comment and suggest changes!

@ibm2431
Copy link
Contributor

ibm2431 commented Jun 24, 2019

but with a twist. The higher the difference between user skill and fish max skill needed, lower the chance to skill up.

Why?

@nuno-pires-cruz
Copy link
Contributor Author

I checked fishing_fish table, there's a total of 133 fishes. of those, about half, have min = 0 and max > 30, meaning, a player with skill 0 can catch a level 30+ fish, so he would just need to find 3-4 types of fishes, to get to skill 100. which is what this thread on BG demonstrates.

the only fishes that have min > 0 , are :
Moat Carp min = 5 , Crayfish min = 2 and legendary min = 90

all guides seem to know the cap level, but not starting level to catch them.

the formula is a proposal. so players pick gaps of 5-10 levels for higher chance of skill up. they can pick higher gaps, but at the expense of chance to skill up.

@TeoTwawki
Copy link
Member

TeoTwawki commented Jun 25, 2019

We only care about emulating what retail actually does, officially. If retail does that, then we want to. if that's not what retail does, then we don't. Acceptable departures are rare and need to have a discussion with strong explanation of why.

@ibm2431
Copy link
Contributor

ibm2431 commented Jun 25, 2019

When I first started going after Mercanbaligi (cap 86) at level 54 compared to when I reached level 56. According to timestamps, those two levels took roughly 16 minutes.

When I first started going after Armored Pisces (cap 108) at level 95 compared to when to when I reached level 96. According to those timestamps, that one level took me 52 minutes.

@nuno-pires-cruz
Copy link
Contributor Author

ibm, that was on retail? If so, it's the opposite of my formula. I can change it to get similar behaviour.

@ibm2431
Copy link
Contributor

ibm2431 commented Jun 25, 2019

Yeah, was on retail (it would be nice if DSP had RoE though!). I might be able to dig through my old logs at some point for harder data, but don't hold me to that - I think that was around 2014, and Logger may or may not have started dropping timestamps from logs / doing the line duplication thing at the time. Those screenshots were just when I saw double-skills.

edit: Actually, if I had the Total Catches RoE set, I guess missing timestamps / duplicated lines are irrelevant; just see how many catches it took to reach a given level in comparison to what I was catching.

whasf
whasf previously requested changes Jul 1, 2019
src/map/utils/fishingutils.cpp Show resolved Hide resolved
src/map/utils/fishingutils.cpp Outdated Show resolved Hide resolved
src/map/utils/fishingutils.cpp Outdated Show resolved Hide resolved
src/map/map.cpp Outdated Show resolved Hide resolved
for (uint8 i = 0; i < 4; i++)
{
random = dsprand::GetRandomNumber(1.);
#ifdef _DSP_SYNTH_DEBUG_MESSAGES_
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought you already removed these /shrug

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not really important, just noticing coz Nyu made me look. 😛

@ibm2431
Copy link
Contributor

ibm2431 commented Sep 3, 2019

Saw this thread pop up in my notifications. Sorry I forgot about it.

My chatlogs on retail from 2015/09/12 to 2015/09/15, during which I took Fishing from level 18 to level 100.

Note: This was during a skill-up campaign. I was also using Fisherman Feast and Pelican Ring.

I have edited the logs with some reg-ex to tag all the fishing-related messages (by type), and also remove non-fishing messages (linkshell, tell, etc). The only lines remaining should be related to fishing, but if any slipped past, I apologize.

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

Successfully merging this pull request may close these issues.

4 participants