Skip to content
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

[extra-dev] Add coq-mathcomp-reals, coq-mathcomp-altreals, coq-mathcomp-reals-stdlib and coq-mathcomp-analysis-stdlib #3192

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

proux01
Copy link
Contributor

@proux01 proux01 commented Oct 29, 2024

ci-skip: coq-mathcomp-classical.dev coq-mathcomp-reals.dev coq-mathcomp-reals-stdlib.dev coq-mathcomp-analysis.dev

…mp-reals-stdlib and coq-mathcomp-analysis-stdlib
proux01 added a commit to proux01/analysis that referenced this pull request Oct 29, 2024
proux01 added a commit to math-comp/analysis that referenced this pull request Oct 29, 2024
@proux01
Copy link
Contributor Author

proux01 commented Oct 30, 2024

Failures are unrelated (a network error and maybe a missing lower bound on ocaml for coq-elpi.dev) let's merge

@proux01 proux01 merged commit 4ce4dbe into coq:master Oct 30, 2024
2 of 5 checks passed
@proux01 proux01 deleted the analysis-reals branch October 30, 2024 06:54
@palmskog
Copy link
Contributor

General comment: I think *-altreals is not a great name for a package, since the name gives no hints on how it's different from *-reals.

@proux01
Copy link
Contributor Author

proux01 commented Oct 30, 2024

Indeed Cc @affeldt-aist I don't know what could be a good name. Anyway, it's currently the remainings of an old uncompleted experiment so it's probably not worth anything for many people. Maybe we could name it something like coq-mathcomp-experimental-altreals or coq-mathcomp-unfinished-altreals, at least until maybe we do something out of it.

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.

3 participants