inbo/camtrapdp

Add taxonomy() accessory function

damianooldoni opened this issue ยท 2 comments

This function returns the taxonomic information from observations(x).

Two questions:

  1. Should we name this taxa() instead?
  2. Should we drop the taxon. prefix for the columns returned by this function?

I'm tempted to say yes to both.

  1. About function name: I understand why you prefer taxa(): it's a plural name as locations(), observations()etc. So, I agree ๐Ÿ‘
  2. In camtraptor's read_camtrap_dp() no prefixes were used, so removing the prefix taxon. will make our life while refactoring camtraptor way easier ๐Ÿ˜„ The point: do we easily can identify these columns and how likely there will be conflicts among taxonomic related columns and observations original columns in the future? I think it will be very unlikely. So, ๐Ÿ‘ to remove the prefix.