ho-dev / HattrickOrganizer

Assistant for Hattrick online football manager
https://ho-dev.github.io/HattrickOrganizer/
GNU Lesser General Public License v3.0
195 stars 79 forks source link

Improve usability team analyzer #66

Closed ArneGloe closed 4 years ago

ArneGloe commented 5 years ago

Here is the issue as requested in the hattrick forum. Thanks for your work! Really appreciate it!

A great add on for me would be to save modified ratings of my next opponent. I often modify these according to my expectations while testing different line-ups in the team analyzer. Right now I manually reset them to my expectations every time I change the line-up.

wsbrenk commented 4 years ago

grafik

my idea is to add an extra row "adjusted" beneath the average row ("Durchschnitt"), when the user changes the opponents ratings in the MatchPredictionPanel. This row could be selected when user changes his lineup to simulate again with his adjusted match ratings.

akasolace commented 4 years ago

@wsbrenk ok yes and how would the reset be done?

wsbrenk commented 4 years ago

@akasolace as long as the opponent team is not changed no reset is needed, i think. if opponent team is changed the "adjusted" row will disappear, until the ratings of the new opponent team is adjusted. then the new adjusted ratings will be added to the list.

as long as the opponent is not changed the user has the selection of using the average, the adjusted or any match rating of the last matches in the match prediction panel. if the user changes again the match ratings, these new values are stored in the adjusted row.

akasolace commented 4 years ago

@wsbrenk ok sounds good !

wsbrenk commented 4 years ago

progress report: it was a hard fight until i understood what is happening where. so i did a little refactoring of the code and i hope other developers confirms that it will get easier to understand. currently the ui looks like this - there is a new "adjusted" row after simulation was run with changed rating or tactic values. grafik

currently i'm looking for the correct match type icon which should be copied from the adjusted match or empty - but not a wrong cup match icon. And there is a lot of testing, which has to be done.

akasolace commented 4 years ago

@wsbrenk I followed this one from far... but thank you for the status update !

akasolace commented 4 years ago

might be implemented by ebc278f

@wsbrenk I will let you close the issue once tested and confirmed to work

wsbrenk commented 4 years ago

@akasolace, @ArneGloe we should try to get the author of the ticket for a test when a new dev version is in place

wsbrenk commented 4 years ago

@akasolace i recognize that i worked longer than a month on this ticket - i'm really tired of this;-)

ArneGloe commented 4 years ago

@wsbrenk: thanks for your work! Really appreciate it. I'm happy to test it as soon as I can. Do you want me to do anaything specific? @akasolace: also to you thanks for your work. Fantastic to see how the project revived the good old HO.

akasolace commented 4 years ago

thank @wsbrenk for the work, that was a big one !

@ArneGloe I will let you know as soon as we have new artefacts that you can test. We have made some heavy changes on the build system and it might take some time before we are back up and running ...

akasolace commented 4 years ago

@ArneGloe hello you can test using https://github.com/akasolace/HO/releases/tag/dev if you are on windows I suggest you to test using HO-4.0.0.2948-win-DEV-JRE.zip In any case please recall it is a dev version and that you should backup your db.

akasolace commented 4 years ago

@wsbrenk I made some test myself and it looks pretty neat !

ArneGloe commented 4 years ago

I just downloaded the 4.0 version and played around a little bit. @wsbrenk that's exactly what I have been looking for. Really cool feature. Thanks again @akasolace I also played around with the rating prediction. With the new rating prediction type, team confidence has no effect on my attack ratings. There might be a problem. The prediction of the defense ratings improved a lot though.

akasolace commented 4 years ago

@ArneGloe thanks or the tests and the feedback .... I will check that

akasolace commented 4 years ago

@ArneGloe this has been fixed(#570)