Zulip Chat Archive

Stream: PR reviews

Topic: !4#3204 simps bug (Grothendieck construction)


Scott Morrison (Apr 04 2023 at 01:26):

@Floris van Doorn, in !4#3204, @[simps] is incorrectly producing lemmas for Prop-valued projections. I can't find the spot in the implementation of simp where it is meant to be discarding these, so I haven't fixed it. Would you be able to take a look at this?

Floris van Doorn (Apr 04 2023 at 11:47):

For fields that are sometimes propositions and sometimes not, @[simps] for now always generates this lemma. I can add a check that also checks whether the equality is propositional just before generating the simp lemma.
(the check for propositional fields is in the code for initialize_simps_projections)

Scott Morrison (Apr 04 2023 at 23:30):

Seems nice to do, otherwise this is a weird cause of simpNF linting errors, which are a pain to diagnose/review.

Scott Morrison (Apr 05 2023 at 01:09):

@Floris van Doorn, I guess a separate request about simp: could we have a way to say "produce all projects except the following", e.g. something like @[simps -inverse_map_fiber_down_down]. When there is just one to remove it is annoying to have to read through the output of @[simps?], and collecting the complement! Also hard to understand afterwards.


Last updated: Dec 20 2023 at 11:08 UTC