coq-robot icon indicating copy to clipboard operation
coq-robot copied to clipboard

reintroduce a type for angle (wip)

Open affeldt-aist opened this issue 3 years ago • 8 comments

affeldt-aist avatar Feb 18 '22 11:02 affeldt-aist

I did not read the PR but one should ideally show that the type angle whatever its definition is a ring quotient of R by the ideal 2 * pi * R

CohenCyril avatar Feb 18 '22 11:02 CohenCyril

I did not read the PR but one should ideally show that the type angle whatever its definition is a ring quotient of R by the ideal 2pi

Right but using which tooling from MathComp?

affeldt-aist avatar Feb 18 '22 11:02 affeldt-aist

I did not read the PR but one should ideally show that the type angle whatever its definition is a ring quotient of R by the ideal 2pi

Right but using which tooling from MathComp?

ring_quotient

CohenCyril avatar Feb 18 '22 11:02 CohenCyril

could it be the definition of angle?

thery avatar Feb 18 '22 11:02 thery

Be careful, you do not need to build angle using a quotient (maybe the arguments of complex numbers was actually the right one), but you can provide a quotient interface and deduce the ring structure from it

CohenCyril avatar Feb 18 '22 11:02 CohenCyril

It could also be the definition of angle... sure

CohenCyril avatar Feb 18 '22 11:02 CohenCyril

This PR is the consequence of an effort to replace the coq-robot definition of trigonometric functions with what is now available in MathComp-Analysis but maybe we should first generalize expR to complex numbers to prove the relation with cos/sin to redefine the already-existing angle as a subtype of R that can finally be endowed afterwards with a ring structure using ring_quotient.

affeldt-aist avatar Feb 19 '22 05:02 affeldt-aist

Reintroducing the type angle makes life a bit more complicated. Before cos and sin were derived from angle, so we had formula for sin(a + b) directly on angle..... Now, sin is defined on R, and when writting sin(a + b) wirh a and b are of type angle, the + is the addition on angle. I don't know the way to go: lifting all the theorem from R to angle or disabling the Zmodule for angle.

thery avatar Mar 01 '22 10:03 thery