chore(FunctionField): rename vars when constant field is not nec. finite#37894
chore(FunctionField): rename vars when constant field is not nec. finite#37894xgenereux wants to merge 4 commits intoleanprover-community:masterfrom
Conversation
PR summary 05953fe9ceImport changes for modified filesNo significant changes to the import graph Import changes for all files
|
| def _root_.FunctionField.FqtInfty := UniformSpace.Completion (RatFunc Fq) | ||
| deriving Field, Algebra (RatFunc Fq), Coe (RatFunc Fq), Inhabited | ||
| /-- The completion `F((t⁻¹))` of `F(t)` with respect to the valuation at infinity. -/ | ||
| def _root_.FunctionField.FqtInfty := UniformSpace.Completion (RatFunc F) |
There was a problem hiding this comment.
Similarly here. InftyCompletionRatFunc?
There was a problem hiding this comment.
But then this shouldn't be in this file right? I feel like the InftyValuation should be in the RatFunc folder no?
There was a problem hiding this comment.
I guess so. (And at some point, it would make sense to move the current content of this file out of NumberTheory into FieldTheory. I think this place should be for material that is specifically about function fields over finite fields. But don't feel obliged to do that now, too!)
There was a problem hiding this comment.
Would you like me to do this move in this PR or in another?
I am asking because all of these lemmas will have to change name (out of the FunctionField namespace) and will need to be deprecated.
MichaelStollBayreuth
left a comment
There was a problem hiding this comment.
Looks good, modulo a few fixes in docstrings and fixing some names.
Asked by @MichaelStollBayreuth here.
I've also slightly adjusted the doc to reflect this.
I used the notation which I know from Rosen - Number Theory in Function Field.