HopkinsIDD / flepiMoP

The Flexible Epidemic Modeling Pipeline
https://flepimop.org
GNU General Public License v3.0
9 stars 4 forks source link

Refine and prune GitHub issue labels #319

Open emprzy opened 1 month ago

emprzy commented 1 month ago

Label

enhancement, meta/workflow

Priority Label

low priority

Is your feature request related to a problem? Please describe.

I've noticed a lack of clarity and some degree of redundancy in the GitHub issue labels. There's a lot of different issue labels, not all are defined clearly, and a few of them are redundant/barely used. I think our workflow would benefit from a more standardized set of issue labels, where it is clear when one should be used.

Second thing: @TimothyWillard has qualms with how the issue templates auto-tag issues with labels based on the contents of the issue (e.g., if the issue contains the word "documentation", the labeler will automatically assign that issue the 'documentation' label). This can create issues if you use a word in your issue that isn't actually the sole focus of the problem (e.g., an issue receiving the 'meta/workflow' label because it contained the word "metadata"). This feature was originally created per request from @shauntruelove in an attempt to make sure people label their issues. I don't really have a dog in this fight, but if Shaun agrees with Tim I will remove the auto-tagging feature.

Is your feature request related to a new application, scenario round, pathogen? Please describe.

No response

Describe the solution you'd like

I've created a document here where I've gone through and refined each of the brief definitions for the issue labels, and also marked the ones that I think could be removed. Here is the synopsis of the labels that I think could be pruned:

The removal of issue tags is the primary thing I want input on. Unfortunately I cannot make a PR with my proposed changes because GH issues are repo-wide, but when we come to a consensus within this thread I'll just implement the desired changes. Requesting input from @shauntruelove, @TimothyWillard , @jcblemai

emprzy commented 1 month ago

Okay actually I do have a dog in this fight now ...the absurd number of issue labels that the auto-tagger gave this issue is evidence that the auto-tagger is perhaps a bunk feature.

Granted, I did explicitly reference the different labels by name so maybe that's not fair but still...

TimothyWillard commented 1 month ago

Okay actually I do have a dog in this fight now ...the absurd number of issue labels that the auto-tagger gave this issue is evidence that the auto-tagger is perhaps a bunk feature.

Granted, I did explicitly reference the different labels by name so maybe that's not fair but still...

That's funny that this issue became a great example of my qualm with the auto-labeler.

But in response to the issue labels:

And I largely agree with the others.

shauntruelove commented 1 month ago

I agree with @TimothyWillard. Keep duplicate and performance, remove the others.

shauntruelove commented 1 month ago

and also agree with removing the auto-assign. seems problematic.

TimothyWillard commented 1 month ago

What are the thoughts on adding a "dependency" label or similar for issues with package dependencies. I'm hoping these issues will decrease with frequency as we continue to stabilize the project, but it seems like the these issues are pretty frequent now:

jcblemai commented 1 month ago

I agree (though this could be paired with the installation label). Performance and Duplicate would be/are really useful

pearsonca commented 1 month ago

would issue labels by language be useful? e.g. language-py, language-R (others as appropriate)

TimothyWillard commented 2 weeks ago

would issue labels by language be useful? e.g. language-py, language-R (others as appropriate)

I think having package specific labels would cover most of this use case? I guess not for scripts that live outside a package.

Also, what are the thoughts on a plotting or viz label? Thinking of:

And others: https://github.com/HopkinsIDD/flepiMoP/issues?q=is%3Aissue+is%3Aopen+%28plot+OR+viz%29.

saraloo commented 2 weeks ago

Yeah I wasn't quite sure labeling when I put that issue together. It might be worth a label for the part of the config it refers to (ie there's no current outcomes label afaik). Plotting/viz might be helpful to label if it's like a check, or post simulation thing? ie preprocessing or postprocessing?

emprzy commented 1 week ago

Okay I just want to summarize before I make these changes, @TimothyWillard @jcblemai @saraloo

Is this correct?

emprzy commented 5 days ago

Update: Just added the new labels and made the necessary changes in the issue templates. New labels are: