Jump to content
Eternal Lands Official Forums
Fedora

Astrology GUI

Recommended Posts

Was this the bug fixed with schmurk's recent CVS commit?

"- corrected a bug that was displaying all the lines of the interface

with a size of 2 pixels instead of 1 when the astrology indicator window

was opened"

If so we can close the Mantis report .

Share this post


Link to post
Share on other sites

No, nothing to do with that. The bug I've corrected is that when you have this window opened, all the other lines on the client interface take a width of 2 pixels instead of 1 because there was missing a glPointSize(1.0) at the end of the function that displays the content of this window.

Share this post


Link to post
Share on other sites

No, nothing to do with that. The bug I've corrected is that when you have this window opened, all the other lines on the client interface take a width of 2 pixels instead of 1 because there was missing a glPointSize(1.0) at the end of the function that displays the content of this window.

Shame, I wouldn't know because I don't use the feature. :icon13:

Share this post


Link to post
Share on other sites

I have the same bug.

 

This one using the el_patch.exe:

gallery_13655_5_2159.jpg

 

And this one using the el_pre_rc_24_02_07.exe:

gallery_13655_5_3274.jpg

 

It only seems to happen with the Acc/magic indicator.

Share this post


Link to post
Share on other sites

Here a small hint:

...

4. Accuracy.**

...

** the accuracy bonus is based on your attack level; the higher the level, the more bonus you get, capped at 5 or so.

and the code astrology.c line 326 & 365:

float calculate_width_coefficient(int amplitude,int value,int invert)
{
float Result = ((float)value / (float)amplitude);
...
}
...
			case atAccMagicIndicator:
			{
				coefficient1 = calculate_width_coefficient(your_info.attack_skill.base/20,value1,0);
				coefficient2 = calculate_width_coefficient(9,value2,0);
			}break;
...
			case atAccPredictor:
			case atAttPredictor: 
			{
				coefficient1 = calculate_width_coefficient(your_info.attack_skill.base/20,value1,0);
				coefficient2 = calculate_width_coefficient(your_info.attack_skill.base/20,value2,0);
				coefficient3 = calculate_width_coefficient(your_info.attack_skill.base/20,value3,0);
			}break;

so people with att < 100 (or 90?) can get this bug (if the cap is 5).

 

is the server calculating too high values for low att level players?

i.e.: should it be possible for an att level 20 player to get +/-5 acc?

Share this post


Link to post
Share on other sites

Only the server knows the right formula to compute the accuracy according to the attack level. The only thing we know is that it should be capped to 5 so let's use this value. And if a player with a low attack never reach this number, does it really matter? The only thing to be sure is that it never goes over 5 though...

Can you confirm this Ent?

Share this post


Link to post
Share on other sites

I've just changed the code to limit the bar length if it would go outside the window. I also report the problem to the error log file. Not a fix but at least it doesn't look stupid now.

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

  • Recently Browsing   0 members

    No registered users viewing this page.

×