Виталий Пухов

Тут нужно определить критерии "плохой\хороший", потому как по графикам не особо заметно разницы, нужно понимать, что если результат не очевиден человеку, объяснить это компьютеру будет еще тяжелее.
Если таких критериев нет нужно их найти, что можно следать:

1. Отобрать как можно больше "плохих" графиков и отдельно "хороших"
2. Нормализовать, то есть привести их виду, где минимальное и максимальное значение по осям будет одно и тоже (если конечно нет критерия вида много по Y - плохо и т.п.)
3. На нормализованной плоскости построить сплайн или просто сразу апроксимировать функции таким образом чтобы получить для каждого графика одинаковый набор значений y для одних и тех же X, к примеру на нормализованной плоскости через каждые n единиц по X получить значение Y.
На выходе получится матрица Xmax*Ymax, где каждой на пересечении будет 1 либо 0
Данные матрицы можно использовать для обучения нейронной сети.

или матрицы не строить а оставить в виде последовательности значений Y, потом сравнивать вектора "плохих" графиков с векторами "хороших",
для сравнения можно будет использовать все что в голову взбредет, то есть любые математические функции, какая ни будь даст достаточно статистически различимый результат. Кстати полученное на п.2-3 можно использовать и для кластеризации.

В принципе в чем затык мне понятно, при изменении коллекции все нормальные foreach или тп. отваливаются, тут либо while либо for с "фичей", к примеру можно так (пример удаление всех нечетных элементов списка):

Random rnd = new Random();
            var s = new List<int>();
            for (int i = 0; i < 20; i++)
            {
                s.Add(rnd.Next(10));
            }
            for (int i = 0; i < s.Count ; i++)
            {
                if (s[i]%2==0)
                {
                    s.RemoveAt(i);
                    i--; //фишка в том чтобы сдвинуть i на единицу назад, т.к. после удаления следующий элемент будет на том же индексе что и старый (удаленный)
                }
            }
            Console.WriteLine(s);

Зависит от количества экземпляров этого класса и чем они кроме этого занимаются, и от необходимой точности. Потому как таймер это не RealTime поток, его событие произойдет не "именно" через 5 минут, а через 5 минут +"как развиснет поток", поэтому если таймеры будут работать в режиме "изменился" раз в 4 мс к примеру и их будет скажем 10, то через пару часов количество их срабатываний разойдется прилично, то есть никакой синхронизации не будет.
В простейшем случае, где нет нужды в синхронизации и их будет мало, проще всего будет запустить Thread в котором будет Sleep(указать время) и вызов Event из инстанса класса, на эти Eventы подпишется основная форма и их обработка будет в потоке интерфейса.

какой смысл с отдельного потока в консоли? Чтобы UI не тормозил?)
единственное что тут может быть это GC не успевает собирать мусор(ибо его будет много)
Можно добавить после завершения блока using код GC.Collect();

Насколько я понял это парсер страниц какого то сайта с поддержкой нескольких прокси серверов, спарсенное складируется в БД, особых костылей в коде не вижу, все достаточно просто и относительно понятно. Из того что заметил странно выглядит обилие репортов от воркера, даже в тех местах что отрабатываются за доли секунды, если по репорту идет обновление UI потока это может замедлить работу. Для читабельности убрал бы лишние комменты (старый код) и лишние переносы строк (пустые строки). Еще отдельный момент обилие messagebox, в работе вероятно нужно постоянно тыкать "ОК" чтобы все продолжало работать, я бы это заменил на вывод в лог или в отдельное поле в UI, кому надо тот сам посмотрит, рабочее приложение в этом обычно не нуждается.

1. Работать будет

но могут быть проблемы с вторичным использованием формы добавления, к примеру если захочется форму 3 и на ней будет уже dataView2 то работать не будет

2. Можно, но не думаю что это нужно,

советую читать в сторону Events, в двух словах, на форме 2 создается событие с параметрами, которые нужно передать в первую форму, первая форма подписывается на нее и делает с этими параметрами что ее душе угодно, вторая форма когда нужно вызывает event и передает параметры. Такая реализация может иметь смысл если в диалоге нужно например выбрать несколько записей или параметров.

Вариант 3 (проще)

Создать параметры на 2 форме (гуглить по "C# get;set;"), соответственно на второй форме они заполняются
На первой форме вызывается ShowDialog и в зависимости от результата просто читаются эти параметры из формы.

Код был явно с недосыпу:) Первое что нужно это почитать о том как вообще делится доступ, то есть гуглить что такое ACL. После осознания истинны можно посмотреть исходник например тут и на основе этого кода написать свою процедуру проверки доступа, после уже чтото писать в лог. Судя по текущему его состоянию ему вообще пофиг есть доступ или нет, он все равно в лог пишет.

Перехватывать глобально, гуглить по "keyboard\mouse hook C#", например тут

В целом разобрался, там в классе еще делегаты были, и если на них подписана форма то шарик хочет ее сериализовать вместе с объектом.