Consider whether tag loss should lead to error by default

epiverse-trace / linelist

R package for handling linelist data

https://epiverse-trace.github.io/linelist/

Other

8 stars 4 forks source link

Consider whether tag loss should lead to error by default #104

Closed Bisaloo closed 6 months ago

Bisaloo commented 8 months ago

I wonder whether the default should be more restrictive by default, to prevent any mishaps from happening. But I guess there's something to be said about having it default to non-blocking behaviors too.

Is this a conscious choice that still stands or is it a legacy decision here?

_Originally posted by @chartgerink in https://github.com/epiverse-trace/linelist/pull/103#discussion_r1482720235_

Bisaloo commented 6 months ago

I believe the new #126 offers a good balance here. By default, linelist will not break any pipelines and use warnings. But users can now more easily change this, even before starting R, with the environment variable.