Repository navigation
Conversation
isSameWorkTitle ראה ב"גרא" וב"הגרא" מילים שונות, ולכן דחה שינוי שם כמו "גר"א על פרקי אבות" ← "הגר"א על פרקי אבות" (otzaria-library#127), והספר איבד את כל כינוייו. עכשיו ה"א שנוספה או נשמטה בתחילת מילה נחשבת כתיב אחר של אותה מילה, כמו כתיב מלא וחסר.
Member
Author
|
@Y-PLONI |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
הבעיה
ב-Otzaria/otzaria-library#127 מוצע להוסיף ה"א הידיעה לשמות ספריא שפותחים בראשי-תיבות של חכם, למשל
גר"א על פרקי אבות←הגר"א על פרקי אבות.הכינויים נשלפים מה-Acronymizer לפי שם הספר, ובשמות החדשים אין כינויים.
carryAcronymsAcrossRenamesאמור להעביר אליהם את כינויי השם הישן, אבלisSameWorkTitleראה ב-גראוב-הגראמילים שונות ודחה את ההעברה ("the title changed words"). כך ארבעת ספרי הפיילוט היו מאבדים את כל כינוייהם, ובאפליקציהגרא אבותו-ראש התורההיו יורדים מכינוי מדויק (דרגה 3) להתאמה מקורבת (דרגה 6). Y-PLONI מצא את זה בדיון ב-kdroidFilter#127.התיקון
ב-
isSameWorkTitle, מילה שנוספה לה או נשמטה ממנה ה"א בתחילתה נחשבת כתיב אחר של אותה מילה, כמו כתיב מלא וחסר. שאר הבדיקות בהעברה לא השתנו: שם ישן שהוא היום שם של ספר אחר, או ששני ספרים יורשים אותו, עדיין נדחה.בדיקות
renamesThatOnlyAddDropOrRespellWordsAreTheSameWork. בלי התיקון הטסט נכשל (גר"א על סדר עולם רבה → הגר"א על סדר עולם רבה), ועם התיקוןAcronymsFollowRenamesTestעובר (18 טסטים).:otzariasqlite:jvmTestהמלא: 138 טסטים, 7 נכשלים. אותם 7 נכשלים גם בענף הבסיסotzaria, כולם ב-OtzariaBuildFailClosedTest, ואינם קשורים לשינוי.אין PR פתוח שנוגע ב-
RenamedBookTitles.kt.קשור: Otzaria/otzaria-library#127, Otzaria/otzaria#2585.