Skip to content

Conversation

Copy link

Copilot AI commented Oct 12, 2025

Problem

The PRZEWODNIK.md file contained incorrect specialization IDs in command examples, causing errors when users tried to follow the guide. Commands like python3 graf_tools.py path ai failed because the actual specialization ID is ai_data_science.

Additionally, the guide lacked clear step-by-step instructions for students and coordinators on how to effectively use the graf_tools.py script.

Changes

1. Corrected All Specialization Command Examples

Before:

python3 graf_tools.py path ai              # ❌ Error: Unknown specialization
python3 graf_tools.py path gaming          # ❌ Non-existent
python3 graf_tools.py path networking      # ❌ Wrong ID

After:

python3 graf_tools.py path ai_data_science    # ✅ Works correctly
python3 graf_tools.py path web_networking     # ✅ Works correctly
python3 graf_tools.py path embedded_systems   # ✅ Works correctly
python3 graf_tools.py path software_engineering
python3 graf_tools.py path signal_processing

2. Updated AI Specialization Example Output

Replaced the minimal, outdated example output with complete, accurate results from ai_data_science, showing:

  • All relevant semesters (1, 2, 3, 5)
  • Current hierarchical format: Semestr → Przedmiot → Koncept
  • Enhanced interpretation explaining what students should focus on in each semester

3. Enhanced Step-by-Step Guidance

For Students:

  • Restructured into clear 3-step process (Krok 1, 2, 3)
  • Added explanations for what each command does
  • Added guidance on how to interpret results

For Coordinators:

  • Added two methods for checking student prerequisites (manual + automatic)
  • Expanded "Adding New Specialization" from 3 to 6 detailed steps with validation
  • Added practical example using the specializations command

4. Added New Practical Examples

  • Web development specialization example showing web_networking path
  • Coordinator example demonstrating how to help students explore options

Verification

All commands have been tested and verified working:

  • python3 graf_tools.py specializations
  • python3 graf_tools.py path ai_data_science
  • python3 graf_tools.py path web_networking
  • python3 graf_tools.py path embedded_systems
  • python3 graf_tools.py path software_engineering
  • python3 graf_tools.py path signal_processing
  • python3 graf_tools.py validate

Fixes #13

Original prompt

This section details on the original issue you should resolve

<issue_title>Popraw treść w pliku Przewodnik.md</issue_title>
<issue_description>@Copilt popraw treść w pliku Przewodnik.md, przede wszystkim tam, gdzie są wywołania komendy 'python3 graf_tools.py path ai' wstaw 'python3 graf_tools.py path ai_data_science' i adekwatnie popraw przykładowe wyniki jej działania zapisane w tym pliku. Adekwatnie popraw pozostałe wywołania komendy zaczynającej się od 'python3 graf_tools.py path'. Skup się na tym, aby krok po kroku pokazać potencjalnym użytkownikom, w tym przypadku studentom i koordynatorom programów dydaktycznych, jak skorzystać z dostępnych możliwości skryptu 'graf_tools.py'.

Originally posted by @pierzcham in #13 (comment)
</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #14


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI changed the title [WIP] Update content in Przewodnik.md for graf_tools.py usage Fix command examples and improve step-by-step guidance in PRZEWODNIK.md Oct 12, 2025
Copilot AI requested a review from pierzcham October 12, 2025 14:39
@pierzcham pierzcham self-requested a review October 12, 2025 14:57
Copy link
Member

@pierzcham pierzcham left a comment

Choose a reason for hiding this comment

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

ok

@pierzcham
Copy link
Member

@copilot merge this PR with the base branch

Copy link
Author

Copilot AI commented Oct 12, 2025

@copilot merge this PR with the base branch

I don't have the ability to merge PRs directly. The PR will need to be merged by someone with repository write access using the GitHub interface or the gh command line tool.

The changes are complete and all tests have passed - the PR is ready to be merged.

@pierzcham pierzcham marked this pull request as ready for review October 12, 2025 15:08
@pierzcham pierzcham merged commit 46f9b80 into copilot/update-syllabus-content-and-format Oct 12, 2025
1 check 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.

2 participants