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

[eFEAT] Insert foxtrick "psicotsi" data #235

Open
mistermax80 opened this issue Sep 30, 2019 · 14 comments
Open

[eFEAT] Insert foxtrick "psicotsi" data #235

mistermax80 opened this issue Sep 30, 2019 · 14 comments
Assignees
Labels
effort: high High efforts expected and/or strong knowledge of HO required reso: info needed Further information is requested type: feature New features
Milestone

Comments

@mistermax80
Copy link
Collaborator

I like to insert the psicotsi function in HO.
Foxtrick has this function, and It will be nice to have in HO.
@akasolace do you think we can consider this?

@akasolace
Copy link
Collaborator

akasolace commented Sep 30, 2019

Definitely, that’s a very good idea for next version.

  1. We would need to use the server to get data. And then do some analysis. Can server cope with that ? That step will be tackle by collect player's TSI and age as part of the existing feedback plugin. #243
  2. decide how to integrate that additional information

Sent with GitHawk

@mistermax80
Copy link
Collaborator Author

Yes of course!
But I don't understand what server should to do.
Could you explain better, how would you do?

@akasolace
Copy link
Collaborator

@mistermax80 I mean than in order to derive the TSI formula, we would need data. Players information from HO users could be send and store to the server. Once on the server we could do the required analysis.

@akasolace akasolace added effort: high High efforts expected and/or strong knowledge of HO required reso: info needed Further information is requested type: feature New features labels Oct 1, 2019
@akasolace akasolace added this to the Backlog / Someday milestone Oct 1, 2019
@mistermax80
Copy link
Collaborator Author

Ok, so Di you want rewrite the formula?
Firstly, I would like to start from unwritten manual: TSI = (1.03Difesa^3 + 1.03Regia^3 + 1.03Attacco^3 + 1.0Passaggi^3 + 0.84*Cross^3)^2 * Calci^0.5 * Forma^0.5 / 1000

TSI_portiere = 3 * Parate^3.36 * Forma^0.5

Or we can see the code in foxtrick.

What do you think?

Anyway, the server can be do that work!

@yaute74
Copy link
Collaborator

yaute74 commented Oct 1, 2019

In the same way it could be good to add some tools, like on HTLiveSight

@akasolace
Copy link
Collaborator

@mistermax80 I don't think we can use the formula from unwritten manual (e.g. age and status are not even mentionned)

@mistermax80
Copy link
Collaborator Author

mistermax80 commented Oct 1, 2019

@akasolace ok, you say right... and if do we use the foxtrick code?

@akasolace
Copy link
Collaborator

@mistermax80 I see 4 options:

  1. we ask user Chechesa if we can use his formula: https://hattrick.chechesa.com/en

  2. we use formula from unwritten manual but only for player which are neither injured or above 28 yo

  3. we use foxtrick formula if we can find them (it might be same formula as 2)

  4. we derive our own formula

  5. could be a nice long term solution whereas we can start with 1, 2 or 3
    I believe 1) to give fairly good result but no idea if Chechesa will be willing to share, I am not even sure if he still playing hattrick

@DavidatorusF considering you know almost every HT player :-), would you feel comfortable contacting Chechesa and asking him?

@mistermax80
Copy link
Collaborator Author

I like 5 :D sound good!!!!

@DavidatorusF
Copy link
Collaborator

DavidatorusF commented Oct 1, 2019

on HTLS, we have ask to Tasos (HT-Tools creator) to use his tools... even if they are open source

about Psicotsi..nice tool for monoskill player... but now we have multiskill player

subskill seem to be better...
https://github.com/ventouris/hattricktools/blob/master/static/js/subskill.js

I never meet Chechesa... seem to me better that a real developer ask him ^^

@akasolace
Copy link
Collaborator

@DavidatorusF I looked at subskill and it does not look that good.
It is true that this kind of analysis is not that good with multiskill player but it is still better than nothing :-)

@akasolace
Copy link
Collaborator

As @mistermax80 suggested we will simply start to collect player's TSI as part of the existing feedback plugin. Hence we will start accumulating data as soon as 2.0 is release

@wsbrenk
Copy link
Collaborator

wsbrenk commented Jun 9, 2024

New Idea: Add "this" feature to the subskill adjustment dialog by adding current and calculated TSI to this dialog.
By adjusting the subs the user is able to narrow the both tsi numbers.

image

@wsbrenk wsbrenk self-assigned this Jun 9, 2024
@wsbrenk
Copy link
Collaborator

wsbrenk commented Jun 9, 2024

TSI = (1.03Def^3 + 1.03PM^3 + 1.03Sc^3 + 1.0Ps^3 + 0.84*Wg^3)^2 * St^0.5 * Fm^0.5 / 1000

TSIgk = 3 * Gk^3.359 * Fm^0.5

(With Solid=6) (Note: non-existent stands for 0, disastrous starts at 0,01 and goes to 1. And so on, making solid 6,01-7,00, and not like generally accepted from 7 to 7,99)

wsbrenk added a commit to wsbrenk/HO that referenced this issue Jun 13, 2024
wsbrenk added a commit to wsbrenk/HO that referenced this issue Jun 13, 2024
wsbrenk added a commit to wsbrenk/HO that referenced this issue Jun 13, 2024
wsbrenk added a commit to wsbrenk/HO that referenced this issue Jun 15, 2024
wsbrenk added a commit to wsbrenk/HO that referenced this issue Jun 15, 2024
wsbrenk added a commit that referenced this issue Jun 22, 2024
* #235 Calculation of the tsi sub

* #235 Form sub in statistic panel

* #235 Form sub in statistic panel

* #235 PlayerSubskillOffsetDialog disable not available skills

* #235 PlayerSubskillOffsetDialog disable not available skills

* #235 review

* #235 review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort: high High efforts expected and/or strong knowledge of HO required reso: info needed Further information is requested type: feature New features
Projects
None yet
Development

No branches or pull requests

5 participants