AllenNeuralDynamics / dynamic-foraging-task

Bonsai/Harp workflow for Dynamic Foraging with Python GUI for visualization and control
MIT License
5 stars 4 forks source link

Automated issue posting is cluttering the GitHub Issues section #990

Open XX-Yin opened 3 days ago

XX-Yin commented 3 days ago

A lot of issues were automatically posted on the GitHub Issues based on the automatic logging, which makes navigation very difficult. I also posted a similar issue before https://github.com/AllenNeuralDynamics/dynamic-foraging-task/issues/949

Issues associated with this auto posting are:

@hanhou @alexpiet what do you think?

alexpiet commented 3 days ago
XX-Yin commented 3 days ago

The primary source of clutter is from the redundant isses. You can check and most of these issue are of this type which makes it very difficult to browse the issue section.

ellahiltonvano commented 2 days ago

@XX-Yin a helpful thing to do might be to use the project board, rather than the issues tab. I have issues organized in the milestone view so they are easier to view

At this point, its important that we are logging these issues so Alex and I can get a sense of where things are going wrong in the code.

XX-Yin commented 18 hours ago

@XX-Yin a helpful thing to do might be to use the project board, rather than the issues tab. I have issues organized in the milestone view so they are easier to view

At this point, its important that we are logging these issues so Alex and I can get a sense of where things are going wrong in the code.

@ellahiltonvano The project board is helpful.

XX-Yin commented 18 hours ago

I think we can avoid duplication of issues by:

The first two criteria seem relatively straightforward to implement, but I’m unsure how to fetch GitHub issues using Python code, which is a prerequisite for meeting the third criterion.

alexpiet commented 8 hours ago
  • Treating the same issues occurring on different rigs as a single issue.

This already happens

  • Treating issues traced to the same line of code as a single issue.

Code lines are ignored because as code updates happen code lines change. Instead issues are grouped by the issue text.

  • Avoiding duplicate posts if an issue, based on the above criteria, already exists in the GitHub issue section.

This already happens. If the issue already exists, my code comments on the original issue.

XX-Yin commented 7 hours ago
  • Treating the same issues occurring on different rigs as a single issue.

This already happens

  • Treating issues traced to the same line of code as a single issue.

Code lines are ignored because as code updates happen code lines change. Instead issues are grouped by the issue text.

  • Avoiding duplicate posts if an issue, based on the above criteria, already exists in the GitHub issue section.

This already happens. If the issue already exists, my code comments on the original issue.

For the second point, it seems that most duplicates are from the same GUI code with the same code lines (example shown below). So, adding this rule will remove most of the duplicates.

Additionally, we can check line number and error information, or code in that line and error information.

image