With GeoGebra 5.0.513.0 28 November 2018 Java 1.8.0_192-64bit, "most" angles are calculated and shown to be 90°, or multiples thereof, or completely unrelated to the points measured.
For example put points at A(0,0), B(1,0) and C(1,1). The angle(A,C,B) according to Geogebra now is 57.3° which is only slightly completely wrong :)
This must have been introduced by some update of Java/Geogebra in December, as it was working fine previously.
Your case works for me (angle is 45 degrees). Could you share your ggb file too?
GeoGebra 5.0.513.0 28 November 2018 Java 1.8.0_181-64bit
It's a just new file with the three points, as an example.
It happens on also on my Notebook.
(In reply to Tobias C. Berner from comment #2)
Could you test my ggb?
My result: http://freebsd.uzsolt.hu/temp/geogebra_bug.png
It's not an issue with a specific file, but the calculations in general being wrong. Be it new or existing geogebra files.
(In reply to Tobias C. Berner from comment #4)
I can't reproduce it.
(In reply to Tobias C. Berner from comment #4)
Does my ggb file works well at you? Does it display 45 or something else?
I confirm the bug too as described in the bug description.
Zsolt Udvari: observe that you are not testing the same version as us. We are working with GeoGebra 5.0.513.0 28 November 2018 Java 1.8.0_192-64bit, while you are working with GeoGebra 5.0.513.0 28 November 2018 Java 1.8.0_181-64bit. Maybe the distfile changed withouth anyone noticing it: in that case a PORTREVISION bump would be needed.
(In reply to Lorenzo Salvadore from comment #7)
I built the geogebra package yesterday (I'm using quarterly, the package is here: http://freebsd.uzsolt.hu/test_pkg/geogebra-5.0.513.0.txz). I think the distfiles are same (because of math/geogebra/distfile hash).
The differences between my and your system is the java/openjdk8's version. I'm using 8.181.13 and you 8.192.26 (check the different java-version in geogebra's output).
Now I'll test the newest GeoGebra (5.0.518.0).
(In reply to Zsolt Udvari from comment #8)
You're right, sorry, I misread: it's the java version that is different indeed. Then probably you could reproduce the bug using the same java version. But of course, if the new Geogebra version works fine for everyone, that's better.
(In reply to Lorenzo Salvadore from comment #9)
The GG 5.0.513.0 works well with OpenJDK 8.181.13 but works wrong with OpenJDK 8.192.26.
The GG 5.0.518.0 (newest version, package is here: http://freebsd.uzsolt.hu/test_pkg/geogebra-5.0.518.0.txz) works well with OpenJDK 8.181.13 but works wrong with OpenJDK 8.192.26.
Conclusion: maybe the newest OpenJDK has a bug?
I created a "bug report" at Geogebra too (https://help.geogebra.org/topic/bad-angle-creation).
It seems it's problem of OpenJDK - its version or build method, I don't know.
(In reply to Zsolt Udvari from comment #11)
I see the Geogebra port does not build anything: NO_BUILD is defined.
If I understand well how the port is made, you compiled the software and then uploaded the result somewhere. I do not have any experience with compiling java programs, but I guess you used OpenJDK in the build process and maybe if you compiled Geogebra again with the new version of OpenJDK the issue could be solved.
Though, this is absolutely not related to this port. I started noticing that something is fishy with the newest OpenJDK 8. I cannot attach to a running JVM in debug options (mvn process) from Eclipse on Windows anymore. It does work for OpenJDK 7 though. I will try to revert and see wether this works.
(In reply to Lorenzo Salvadore from comment #12)
Sorry, you've right.
The package creation in this case is very simple: copy some files to correct directory (do-install target).
I think there's a general problem with the java.lang.Math, as noted by Thomas Gude in:
Created attachment 200863 [details]
geogebra - Temporary fix for angles
Here is a temporary fix for the bug.
It makes geogebra use openjdk7 instead of openjdk8: I tested successfully the patch.
(In reply to Lorenzo Salvadore from comment #16)
Why don't you downgrade to 181 and say "pkg lock openjdk8"?
Created attachment 200864 [details]
gegoebra - Temporary fix for angles
Sorry, I had inverted the order of the files in the diff command. Now it should be fine.
(In reply to Michael Osipov from comment #17)
Because for new users only last version of openjdk8 is directly available for installation in the ports tree. Openjdk7 instead is easily available for everyone. But indeed, your solution would work as well.
Created attachment 200865 [details]
geogebra - Temporary angles fix
I've added a PORTREVISION bump, in case the patch was to be committed (initially I thought it as a patch to be manually applied, but maybe it is better to committ it, in case the openjdk8 bug is long to correct).
(In reply to Lorenzo Salvadore from comment #20)
If the bug is in openjdk would better to downgrade openjdk8 and increase its PORTEPOCH - or fix its bug :)
If in some days doesn't change anything in bug #234689 should use your patch.