"`rationalize`" a rational

[ParadaCarleton]

It might be useful to have rationalize(x) return x if x is already rational, to allow generic code that takes both reals and rationals as inputs.

[ayushsengupta8]

Hello,
I am completely new to open source projects and would like to try to solve this issue.
I can use a convert() function to rationalize(x), however, it would be helpful if I know in which folder, in which file do I have to change/add the code. @logankilpatrick

[fredrikekre]

As you can see by the link just above your comment someone has already started to work on this.

[ayushsengupta8]

@fredrikekre Oh! Thanks...and sorry for the lame comment

[ndinsmore]

@dkarrasch I was just looking at this issue and it looks like you meant for it to be closed by #43427

[vmpyr]

This issue seems to have been resolved. I think it should be closed now

[vtjnash]

Still fails. (waiting for #43427 to be merged)

julia> rationalize(rationalize(1.0))
ERROR: MethodError: no method matching rationalize(::Rational{Int64})

[vmpyr]

I am sorry, I misinterpreted something. My bad. Is that PR still WIP?

[ParadaCarleton]

I am sorry, I misinterpreted something. My bad. Is that PR still WIP?

I don't think it is, so much as it's finished but hasn't been merged.