JuliaDynamics / RecurrenceAnalysis.jl

Recurrence Quantification Analysis in Julia
https://juliadynamics.github.io/RecurrenceAnalysis.jl/stable/
MIT License
45 stars 12 forks source link

[WIP] rename plotting functions and improve their documentation #62

Closed Datseris closed 5 years ago

Datseris commented 5 years ago

bit more concistent names:

recurrencetext, recurrencescatter, recurrenceplot

also exporting recurrencescatter, because I want to put this to the docs as well.

Closes #58

heliosdrm commented 5 years ago

What about renaming recurrencescatter as recurrencecoordinates ?

Perhaps recurrencetext and recurrenceplot might have more meaningful names as well. recurrencetext is not that much intuitive, and both are "plots" somehow...

Datseris commented 5 years ago

Honestly, I very much dislike the fact that we include recurrence in the names of all these functions. The recurrence part should be implied anyway from the input type.

Can we do better I wonder? Of course we have to avoid names like only scatterplot etc. that may conflict

heliosdrm commented 5 years ago

I was thinking nearly the same while I was coming home. What about:

Datseris commented 5 years ago

My only suggestion is imagematrix instead of image (because it does not produce an image).

heliosdrm commented 5 years ago

Or maybe grayscale?

Datseris commented 5 years ago

That was a lot of work! 👍

Naaah, mostly Atom's FInd All -> Replace All magic :)