-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
start porting to MathComp 2 #131
Conversation
CI needs to be fixed |
The currenty error is:
It looks like a confusion due to the fact that we define isConvexSpace both in analysis and in infotheo |
Commenting out the
|
I didn't try this one yet. Thanks. This is worrisome though. I think that we still want to rename isConvexSpace in infotheo because it is very likely that both definitions of convex space will need to coexist for a while (and/or one will become a factory for the other). Don't you think? |
I am reluctant to rename either of them since they are exactly the same notion and the definitions will eventually converge; the renaming will induce many changes in dependent code, and later renaming back all of them. |
In any case, we'll have to do something about the coexistence of these two definitions. |
fyi @garrigue @t6s