atorus-research / metacore

https://atorus-research.github.io/metacore/
Other
33 stars 7 forks source link

More explicit message: spec_type_to_* #92

Open cpiraux opened 1 year ago

cpiraux commented 1 year ago

image

It could be interesting to add sheet = Variables in the message so it is more clear that this sheet is selected by the function parameter sheet=

image

I suggest adding a list of columns that match, which will enable direct identification of all the possibilities.:

image

We might also add in message “Please check your regular expression” for which function parameter, regular expression should be updated, it is possible for cols = and sheet=. I also find that “Please check your regular expression” may not be very user-friendly for beginner programmers who might not be familiar with regular expressions. My suggestion would be:

image

After reading this message, I initially assumed that only the specified columns needed to be updated and the function would automatically match them correct ones. However, I later realized this was not the case. It would be helpful to include the correct matching of columns in the function output, so that users can verify the values and identify any additional columns that may need to be included in the function call.