search

coin-or / prtpy

Number partitioning in Python
MIT License
46 stars 20 forks source link

pull request #6

Closed edutco closed 1 year ago

edutco commented 2 years ago

please consider to add my epsilon appriximation algorithm to your repository

erelsgl commented 2 years ago

א. האם תוכלי להוסיף את האלגוריתם שלך למקום המתאים כאן: https://github.com/erelsgl/prtpy/blob/main/prtpy/__init__.py ולוודא שהבדיקות עדיין עוברות?

ב. הקובץ של הסביבה הוירטואלית לא אמור להיות ב-PR. גם לא הקובץ של אלגוריתמי אריזה.

edutco commented 2 years ago

הוספתי את שם הקובץ לinit מקווה שעשיתי את זה טוב לא הבנתי מה זה PR ומה זה אלגוריתמי אריזה (הכוונה לתקייה packing? למחוק אותה מאצלי?)

erelsgl commented 2 years ago

ברשימת השינויים שעשית, רואים כמה קבצים שלא קשורים לשינוי עצמו: image לא צריך למחוק את הקבצים, אלא רק לבטל את השינויים שלא קשורים.

אני לא רואה שינוי בקובץ init - האם הגשת ודחפת את השינוי מהמחשב שלך?

erelsgl commented 2 years ago

לא צריך למחוק את הקבצים, אלא רק לבטל את השינויים שלא קשורים.

אני לא רואה שינוי בקובץ init - האם הגשת ודחפת את השינוי מהמחשב שלך?

edutco commented 2 years ago

לא צריך למחוק את הקבצים, אלא רק לבטל את השינויים שלא קשורים.

אני לא רואה שינוי בקובץ init - האם הגשת ודחפת את השינוי מהמחשב שלך?

from prtpy.partitioning.approximation_schemes_AAWY import mainAlgorithm זו השורה שהוספתי אבל אני לא רואה אותה. הוא גם לא נותן לי לערוך את הקובץ שלך.

erelsgl commented 2 years ago

תיכנסי לזום: https://ariel-ac-il.zoom.us/j/82946424408 הסיסמה הרגילה

erelsgl commented 2 years ago

כדאי לתת לזה שם יותר מתאים, למשל:

from prtpy.partitioning.approximation_schemes_AAWY import mainAlgorithm as AAWY_approximation_scheme

את רק צריכה לעשות commit + push מהמחשב שלך, וזה אמור להגיע אוטומטית לכאן.