mv squeeze for R earlier#2004
Conversation
8e82c4d to
74427e4
Compare
|
CI green (fyi: @mkerjean ) |
mkerjean
left a comment
There was a problem hiding this comment.
Looks great, thanks. As the CI is green it can be merged. Could you explain why the necessity to duplicate cvgrPdistC_lt or cvgr_distC_lt ? Would solving it necessitate the creation of a pseudometric type?
By putting a So this is really just a way to make an incremental progress (put the squeeze theorem in the right place and
Sorry, I can't see it. Which type are you thinking of? Pseudometric in the MCA sense or in the sense "d(x,y)=0 does not necessarily imply x=y"? In the latter case, it does not seem to be the incriminating property, that's why I can't see how it would help right away. |
Motivation for this change
This PR is essentially moving
squeeze_cvgrfromnormed_module.vto thetopology_theorydirectoryso that it is available to develop the theory of TVSs.
Checklist
CHANGELOG_UNRELEASED.mdReference: How to document
Merge policy
As a rule of thumb:
all compile are preferentially merged into master.
Reminder to reviewers