Skip to content

fixed comments#38

Open
oriariel wants to merge 7 commits intoerelsgl:masterfrom
oriariel:master
Open

fixed comments#38
oriariel wants to merge 7 commits intoerelsgl:masterfrom
oriariel:master

Conversation

@oriariel
Copy link

No description provided.

@avivdan
Copy link

avivdan commented Jan 22, 2023

אני גם פה

avivdan and others added 3 commits February 1, 2023 13:25
… the courses of them, added test to demand function in algorithm1
… the courses of them, added test to demand function in algorithm1
אחרי שינויים של אראל
graphics card = None
and youtube in the background...
''' No newline at end of file
pytest.main(args=["fairpy/course_allocation"]) No newline at end of file
Copy link
Owner

Choose a reason for hiding this comment

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

כמו שכתבתי לכם כבר (נראה לי): תשנו כאן ל:

pytest.main(args=[__file__])  

וכן בשאר הקבצים. כך שזה יעבוד על כל המחשבים בלי תלות במסלולים.

@@ -0,0 +1,230 @@
'''
Heuristic search algorithm through price space, originally developed in Othman et al. 2010
Copy link
Owner

Choose a reason for hiding this comment

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

יש להוסיף כאן (ובכל שאר הקבצים) את השם המלא של המאמר, שמות המחברים, וקישור למאמר.

import logging


logger = logging.getLogger(__name__)
Copy link
Owner

Choose a reason for hiding this comment

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

כל השורות שמשנות את תצורת הלוג הגלובלית (בקובץ זה שורות 22-28, וכן בשאר הקבצים) לא צריכות להיות ביחידה, אלא רק בתוכנית הראשית.

year=year, courses=courses, preferences=preferences))

max_budget = 25
price_vector1 = adaptors.divide(algorithm1,input=students,courses=courses, max_budget=max_budget, seed=3, time_to=10)
Copy link
Owner

Choose a reason for hiding this comment

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

לא הצלחתי להבין מהדמו שלכם, איפה בדיוק נמצא האלגוריתם שמקבל כקלט את ההעדפות של השחקנים, ומחזיר כפלט את חלוקת הקורסים (איזה קורסים מקבל כל שחקן)?

תסתכלו בשאר הקבצים בספריה, למשל round_robin.py ו round_robin_demo.py ותראו למה הכוונה. הפונקציות של החלוקה מקבלות כקלט מבנה שמכיל את הערכים של השחקנים (איזה ערך הם מייחסים לכל חפץ), ומחזיר כפלט חלוקה של חפצים בין השחקנים (עבור כל שחקן, איזה חפצים הוא קיבל).

Copy link

Choose a reason for hiding this comment

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

preferences = random.sample(courses, random.randint(3, 7))
לקלט של העדפות,
אמרת שלגבי הערך המוחזר אפשר להדפיס בסוף...

Copy link
Owner

Choose a reason for hiding this comment

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

התכוונתי לשאול: איפה יש לכם פונקציה כמו נניח round_robin, שמקבלת כקלט את ההעדפות של השחקנים, ומחזירה כפלט איזה קורסים כל שחקן קיבל?

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