Skip to content

Conversation

@alex98247
Copy link
Contributor

Summary

Remove unused old experiment data

Quick changelog

  • Remove old config examples
  • Remove old generator examples
  • Remove old experiment data
  • Remove old storages

What's new?

@alex98247 alex98247 requested review from f1i3g3 and faux66 October 1, 2025 09:54
@alex98247 alex98247 self-assigned this Oct 1, 2025
Copy link
Contributor

@f1i3g3 f1i3g3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Есть несколько вопросов, которые почему-то не хотят прикрепляться к соответствующим файлам. Расписал ниже.

  1. У нас сейчас есть хоть какой-то пайплайн? Надо, чтобы был.
  2. Мне кажется, что надо оставить варианты экспериментов/переписать их под новую архитектуру. Потому что кто-то, прочитавший ФРУКТовую статью, может захотеть посмотреть исходники.
  3. SQLite все-таки считаю нужным оставить - как пример размещения (я сейчас как раз мучаюсь с авто конфигурацией).
  4. Там еще примеры отчетов потерялись.
  5. Надо еще мой старый пулл реквест глянуть, мб что-то из него можно вытащить (ресолверы, например).

Copy link
Contributor

@f1i3g3 f1i3g3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Есть несколько вопросов, которые почему-то не хотят прикрепляться к соответствующим файлам. Расписал ниже.

  1. У нас сейчас есть хоть какой-то пайплайн? Надо, чтобы был.
  2. Мне кажется, что надо оставить варианты экспериментов/переписать их под новую архитектуру. Потому что кто-то, прочитавший ФРУКТовую статью, может захотеть посмотреть исходники.
  3. SQLite все-таки считаю нужным оставить - как пример размещения (я сейчас как раз мучаюсь с авто конфигурацией).
  4. Там еще примеры отчетов потерялись.
  5. Надо еще мой старый пулл реквест глянуть, мб что-то из него можно вытащить (ресолверы, например).

@f1i3g3
Copy link
Contributor

f1i3g3 commented Oct 1, 2025

Почему-то дважды отправилось, ну да ладно.

@alex98247
Copy link
Contributor Author

Почему-то дважды отправилось, ну да ладно.

Есть несколько вопросов, которые почему-то не хотят прикрепляться к соответствующим файлам. Расписал ниже.

  1. У нас есть пайплайн, просто пайплайны изменились
  2. Мы сделаем шаблоны конфигураций экспериментов, просто таскать этот хлам значит, что на него кто-нибудь завяжется.
  3. SQLite Остается, его никто не выпиливает + подготовил отдельный реквест с тестами на SQLite и добавил Alchemy. Alchemy поддерживает SQLite. Поэтому нет смысла таскать SQLite, если есть более общий стор
  4. А зачем нужны примеры отчетов? Ты думаешь они нужны? Я бы вернул их, но когда подумаем и обсудим всю папку examples + можно их вынести в ридми или корень
  5. Но это же не относится к этому реквесту, мы когда будем добавлять ресолверы натянутые на текущую архитектуру, придумаем заново (с теми ресолверами помню были проблемы)

@faux66
Copy link
Contributor

faux66 commented Oct 2, 2025

Есть несколько вопросов, которые почему-то не хотят прикрепляться к соответствующим файлам. Расписал ниже.

1. У нас сейчас есть хоть какой-то пайплайн? Надо, чтобы был.

2. Мне кажется, что надо оставить варианты экспериментов/переписать их под новую архитектуру. Потому что  кто-то, прочитавший ФРУКТовую статью, может захотеть посмотреть исходники.

3. SQLite все-таки считаю нужным оставить - как пример размещения (я сейчас как раз мучаюсь с авто конфигурацией).

4. Там еще примеры отчетов потерялись.

5. Надо еще мой старый пулл реквест глянуть, мб что-то из него можно вытащить (ресолверы, например).

Все это только в заблуждение вводит и никак не используется, тогда зачем оставлять?
А конфигурации эксперементов когда будут, можно будет добавить и примеры их отчетов.

@faux66 faux66 closed this Oct 2, 2025
@faux66 faux66 reopened this Oct 2, 2025
@f1i3g3
Copy link
Contributor

f1i3g3 commented Oct 3, 2025

Есть несколько вопросов, которые почему-то не хотят прикрепляться к соответствующим файлам. Расписал ниже.

1. У нас сейчас есть хоть какой-то пайплайн? Надо, чтобы был.

2. Мне кажется, что надо оставить варианты экспериментов/переписать их под новую архитектуру. Потому что  кто-то, прочитавший ФРУКТовую статью, может захотеть посмотреть исходники.

3. SQLite все-таки считаю нужным оставить - как пример размещения (я сейчас как раз мучаюсь с авто конфигурацией).

4. Там еще примеры отчетов потерялись.

5. Надо еще мой старый пулл реквест глянуть, мб что-то из него можно вытащить (ресолверы, например).

Все это только в заблуждение вводит и никак не используется, тогда зачем оставлять? А конфигурации эксперементов когда будут, можно будет добавить и примеры их отчетов.

Как я понял, SQLite всё же оставляем как один из вариантов размещения (я как раз решаю эту задачу сейчас) в соседнем PR.
Также считаю, что если мы что-то убираем не совсем (а просто меняем форму), то лучше эту форму сразу и залить.

@f1i3g3
Copy link
Contributor

f1i3g3 commented Oct 10, 2025

Резюме: пока не мерджим, пусть висит.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants