Skip to content

Conversation

@NotBioWaste905
Copy link
Collaborator

Refactored and simplified version of the repository

Key changes:

  1. Renamed project from chatsky_llm_autoconfig to dialogue2graph
  2. Flattened repo: now it is single dialogue2graph project without dev_packages
  3. New project structure:
dialogue2graph
├── datasets
│   ├── complex_dialogues
│   │   └── prompts
│   └── core
├── metrics
│   └── test_cases
└── pipelines
    ├── core
    ├── cycled_graphs
    │   └── prompts
    └── simple_graphs
        └── prompts
  1. New experiments convention (see experiments/README.md for more context)

@NotBioWaste905 NotBioWaste905 self-assigned this Feb 10, 2025
@NotBioWaste905 NotBioWaste905 marked this pull request as draft February 10, 2025 09:17
@NotBioWaste905 NotBioWaste905 requested a review from kudep February 10, 2025 09:17
@NotBioWaste905 NotBioWaste905 marked this pull request as ready for review February 18, 2025 14:18
Copy link
Collaborator

@kudep kudep left a comment

Choose a reason for hiding this comment

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

  • это PR про рефакторинг, если кэширование уже работает - ок, если нет убрать его из PR
  • не понятные файлы examples.ipynb (лучше сделать отдельную директорию с примерами под, генерацию графа, расчет метрик, генерацию данных, генерацию графа с достроением), ideas.md
  • неактульные данные https://github.com/deeppavlov/chatsky-llm-autoconfig/blob/be7fcd4ba3e7577517a9c8f355293ef4868ced15/README.md#L70-L73
  • нет реальных примеров использование, их планируем добавлять?

@NotBioWaste905 NotBioWaste905 merged commit 4b367b7 into dev Feb 21, 2025
4 checks passed
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.

3 participants