Skip to content

Conversation

@affeldt-aist
Copy link
Owner

No description provided.

@affeldt-aist affeldt-aist marked this pull request as draft February 18, 2022 11:07
@CohenCyril
Copy link
Collaborator

CohenCyril commented Feb 18, 2022

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

@affeldt-aist
Copy link
Owner Author

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?

@CohenCyril
Copy link
Collaborator

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

@thery
Copy link
Collaborator

thery commented Feb 18, 2022

could it be the definition of angle?

@CohenCyril
Copy link
Collaborator

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
Copy link
Collaborator

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

@affeldt-aist
Copy link
Owner Author

affeldt-aist commented Feb 19, 2022

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.


Lemma sin_eq0 (r : R) : sin r = 0 <-> exists k, r = k%:~R * pi.
Proof.
split; last by move=> [k ->]; rewrite sin_int_pi.
Copy link
Collaborator

@thery thery Feb 23, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could this also be handled by some general alternating property

f x = 0, alternating f p -> there exists  0 <= x <p /\ f x = 0
``.

Copy link
Owner Author

@affeldt-aist affeldt-aist Feb 23, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your comments. This one I hadn't it time to address it today.
See 2a5197b

@thery
Copy link
Collaborator

thery commented Mar 1, 2022

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants