You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
[Automatically converting 2pi to tau could actually be] counterproductive to the "tauist cause": if one chooses to use the Tau package, it would make sense for them to take the extra step to change their code to explicitly reflect their choice of circle constant, rather than leaving the 2pi's around.
With that in mind, though, I think it would be nice to provide some sort of warning for when a downstream package uses 2pi in its code -- which could be done by overriding *(2, pi), without changing its resulting type. I'm not sure if it's possible to distinguish code from a module from code from modules it imports (we wouldn't want to warn a user of instances of 2pi in packages they didn't author);
If that distinction is not possible (or the warning approach found too intrusive), then an alternative could be a helper script or method in this package that would process a given module or file and output a list of locations where 2pi could be replaced by tau. This is easy enough to do by hand, but depending on the search method, one could easily miss valid instances -- 2pi, 2*pi, 2 * pi, 2.0 * pi, pi*2, etc. -- so an automated process might be useful.
I don't have a strong opinion on this. In any case, I wouldn't make it part of the package itself, at least an extra script to be placed outside src directory.
So this issue it to track the implementation of such a script.
The text was updated successfully, but these errors were encountered:
@waldyrious is this issue still relevant?
I would like to work on it! I've understood the issue from the discussion and also the proposed solution of an extra script.
Could you help me get started?
Hey @cyborg1995 — thanks for the offer, and sorry for the late response. I am afraid I can't help much with the creation of the script at this point, since I haven't worked on this package (or any Julia code, for that matter) for a long time.
That said, if you have specific questions, I'd be happy to attempt to respond or help however I can.
From issue #22:
To which @giordano replied:
So this issue it to track the implementation of such a script.
The text was updated successfully, but these errors were encountered: