Comments (6)
Looking at this error list I can't figure out if there is a number of logical errors or if it's a matter of adding some exception rules.
This one stands out as a possible error in the test string:
- Ŏ Yun-jŏk
? ^
+ Ŏ,Yun-jŏk
? ^
: S2R transliteration error for korean_names!
Original: 어윤적
I haven't seen the comma notation elsewhere. Was that intended?
from scriptshifter.
It is actually a very simple logic.
Func NameRomanizer() checks if there is a comma or a center dot.
If yes, it separates strings by the comma or the center dot and romanizes each name separately: Func BatchRom()
If no, it just romanizes one name: Func KorNameRom20()
from scriptshifter.
The notation you mentioned is for a different situation, where cataloging rules (not romanization rules) require a comma between the last name and the first name. This function should not turn on globally, because it applies only to certain fields like 100, 600, 700, etc.
from scriptshifter.
Right, I was referring to the comma in the result rather than in the source.
So, the resolution for Ŏ,Yun-jŏk
would be to replace the comma in the expected result with a space?
If you think that an extra option may be necessary for adding a comma in specific fields, I can add it. I tried to avoid per-script options so far, but if it's necessary, I'll add it.
from scriptshifter.
Basically, there should not be a comma between the first name and the last name.
The logic was, romanization with no comma (a comma never occurs in Hangul script anyway).
If the Hangul script is in the field 100, 600, 700, or 800, then add a comma following the cataloging practice.
from scriptshifter.
I added an option to specify the MARC field for Korean names. If the value is one of those you indicate, the Romanized text will add a comma after the last name.
from scriptshifter.
Related Issues (20)
- Hebrew HOT 1
- Yiddish HOT 1
- Arabic HOT 1
- Japanese
- Greek HOT 2
- Branching Language List, Favorite a Language via Cookie or Shortcut? HOT 5
- Chinese: parse numerals HOT 1
- Tibetan tests and fixes HOT 1
- Explore database design
- Slim down ML packages
- Admin CLI
- Provide s2r or r2s availability in the GET /languages response HOT 2
- Allow option to enable CORS headers sent in response HOT 1
- Application exceptions not surfacing correctly in UI HOT 1
- Assamese
- closing punctuation converted to alphanunmeric text
- Add MARC codes to language index HOT 1
- Bibframe.org issue HOT 6
- S2R cyrillic omits first letter when capitalize is on HOT 3
- Regular expressions for ignore patterns
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from scriptshifter.