-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Further <ls> cleanup #136
Comments
💯 |
This week's changes to mw.txt, primarily to ls markup, are now completed. The sequence of changes are in files change_1.txt through change_4.txt, with corresponding notes in the readme.txt file of issue136 directory. The change_all.txt file shows list of all 4221 lines changed in mw.txt. The ls markup in mw.txt has received considerable attention in the last few weeks. At the moment, I don't have in mind further lines of improvement to ls markup in mw.txt, and will likely return to ls markup improvements in pw and pwg. |
Probably @funderburkjim may consider removing the 3 change change change |
And also correct the space-between-digits |
Some more minor corrections, on a quick look--
And it is not out of place to mention that many punctuation errors, as at cases 9 & 10 above, are seen throughout the text; and a handful cases of hyphen marks at wrong places which indicate the |
@Andhrabharati Will take a look at these flaws; is there a systematic way to find other instances like 9, 10 ? |
Though these are not related to And, there is a |
further cleanup.These take into account suggestions since previous commit. A large number of 'new' ls abbreviations were added to mwath as 'Unknown'. @Andhrabharati could help by providing tooltips for these Unknown cases. An attempt was made to define programmatically a 'normal' ls instance. Using this rule, If there are no more correction suggestions for 'ls' in mw, this issue can be closed and I'll take |
Seen that the space between digits and other corrections related to The remaining point in this issue is #136 (comment), which could be done before going to #131, or to be kept in mind while doing the I would prefer them being corrected here itself, as these are not marked So Jim can decide the action accordingly to close this issue. |
If @funderburkjim likes to do it here itself, I can surely help resolving these, but I would suggest doing this piece of work while some action is taken on the issue #135 (which is related to the same and also has some more relevant points). Hope to listen back Jim's opinion. |
Some small extra corrections related to spaces:
|
In the tooltip.txt,
to be corrected as
|
There are two instances of Incidentally, the When looked for the equivalence among these 4 files, noticed that both mwauth.txt & tooltip.txt have 168 no.s of to be expanded "Unknown reference" entries, whereas both abbrevlist_unknown.txt & ls_abbrev_instances_unknown.txt listed just 147 no.s. What is the reason for the difference of 21 between the two sets of files? The 21 additional entries in tooltip.txt are--
In these, Śak. (Pi.) occurs 18 times!! |
Resolving 9 out of 10 instances of
|
while you're on this MW work, would you mind generating the IAST version of mw.txt again [so that I can do a better (rather, faster) work using it]? |
2nd batch of corrections.These take into account the preceding comments by @Andhrabharati. Note 1: The one remaining
Note 2:
The iast version of revised mw.txt is temp_mw_6_iast.zip. The unknown ls abbreviations file is revised and contains 170 items: abbrevlist_unknown.txt Instances of the abbreviations with unknown tooltips are in |
We can discuss tooltips for the unknown literary source abbreviations under #135. Perhaps now we can consider this #136 closeable? |
Wonderful updates! About to finish resolving the unknown reference entities (just another 15 remaining). |
And you can close this issue now. |
This issue devoted to continuation (from #134) of cleanup of ls markup in mw.txt.
Two fertile areas:
The text was updated successfully, but these errors were encountered: