Skip to content

Conversation

@tabareau
Copy link

No description provided.

@jdchristensen
Copy link
Collaborator

Here's the link to the Rocq PR: rocq-prover/rocq#21164

Unfortunately, neither this PR nor that PR say what that PR does... @tabareau , can you explain?

Comment on lines 423 to 424
(* This hint is required to be able to use cumulativity of inductive types *)
(* together with first-order approximation for polymorphic hints. *)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't know what "first-order approximation" means in this context.

(Also, minor point, but in Coq-HoTT comments are written as a single comment with no line breaks.)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
(* This hint is required to be able to use cumulativity of inductive types *)
(* together with first-order approximation for polymorphic hints. *)
(* This hint is required to be able to use cumulativity of inductive types together with first-order approximation for polymorphic hints. *)

@tabareau tabareau force-pushed the hint-refresh-firstorder-approx branch from 575f308 to f0d5996 Compare October 20, 2025 04:55
@tabareau
Copy link
Author

Here's the link to the Rocq PR: rocq-prover/rocq#21164

Unfortunately, neither this PR nor that PR say what that PR does... @tabareau , can you explain?

you are right, @ppedrot will explain on the PR.

@jdchristensen
Copy link
Collaborator

Closed in favour of #2318 .

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.

2 participants