Context Navigation

Changes between Initial Version and Version 1 of Ticket #21129, comment 40

initial	v1
1	1	Code itself looks good now. The test I was trying to run also doesn't fail now. Using Prop 4.3 in Birdy and Larson's paper on the pairing, I tested pairing two different dynamical systems, and the results were mathematically correct correct.
2	2
3		The only remaining problem is the example beginning on line 1224. According to Prop 18 of Petsche, Szpiro and Tucker, the value of the pairing should be exactly the global height of a. Thus, this is mathematically incorrect. Since it was passing before the code was modernized, it could have been something in the modernization. Take a look and try and see what could be going wrong, and if you get stuck, this is worth emailing Paul about.
	3	The only remaining problem is the example beginning on line 1224. According to Prop 18 of Petsche, Szpiro and Tucker, the value of the pairing should be exactly the global height of a. Thus, this is mathematically incorrect. Since it was passing before the code was modernized, it could have been something in the modernization. It is also possible that we just need to increase `n`, or change the noise modifier to get this example to work. Take a look and try and see what could be going wrong, and if you get stuck, this is worth emailing Paul about.