Skip to content
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

Suggestion: Frequency of names report: Consider surnames in male and female forms as same #552

Closed
burtek opened this issue Apr 29, 2024 · 11 comments · Fixed by #555
Closed
Assignees
Labels
enhancement New feature or request

Comments

@burtek
Copy link
Contributor

burtek commented Apr 29, 2024

There is a rule in polish language that if the surname ends with -cki/-ski/-dzki, it changes into -cka/-ska/-dzka for females. Not sure if any other language has something like this, but I guess it would be nice to be able to specify such rules for the report generator.

My idea is a list of "surname replacements" for the report generation. In this case, before the report generation (in app settings?), I could provide following replacements (using regex):

  • Find: ck[ai]$, Replace with: cki
  • Find: sk[ai]$, Replace with: ski
  • Find: dzk[ai]$, Replace with: dzki

Report generator would then apply those replacements for each surname as they are counted and instead count based on the replacement output.


NB: I know the frequency of names report is on bottom of the importance list of GEDKeeper modules, so I don't really expect it to be implemented, but I figured I might as well open this issue anyway and just suggest it

@Serg-Norseman Serg-Norseman self-assigned this Apr 29, 2024
@Serg-Norseman Serg-Norseman added the enhancement New feature or request label Apr 29, 2024
@Serg-Norseman Serg-Norseman added this to the Next Versions milestone Apr 29, 2024
@cbettinger
Copy link

Be aware, that the list of replacements you suggested, is not complete. For example the wife of "Konieczny" is "Konieczna".

@burtek
Copy link
Contributor Author

burtek commented May 2, 2024 via email

@Serg-Norseman
Copy link
Owner

Such functionality exists for the Russian language. But it is implemented as part of the culture class code (the NormalizeSurname function). Nothing prevents us from creating an analogue for the Polish language.

https://github.com/Serg-Norseman/GEDKeeper/blob/master/projects/GKCore/GKCore/Cultures/PolishCulture.cs
https://github.com/Serg-Norseman/GEDKeeper/blob/master/projects/GKCore/GKCore/Cultures/RussianCulture.cs

@burtek
Copy link
Contributor Author

burtek commented May 2, 2024

That works, too. I can and come up with a PR based on the russian code.

Does that mean though, the behaviour will depend on the language set by the user?

@Serg-Norseman
Copy link
Owner

Yes

@burtek
Copy link
Contributor Author

burtek commented May 2, 2024

So that means it won't work for me if I'm using English language... :/

@Serg-Norseman
Copy link
Owner

Cultures are tied not to the interface language, but to the language of the database (File\Properties) or to the language of each individual person name.

@burtek
Copy link
Contributor Author

burtek commented May 3, 2024

Oh, gotcha. Makes more sense. Thanks.

@Serg-Norseman
Copy link
Owner

You improved localization into Polish and did this PR. According to my principles, I must mention you in the history of the project. Tell me, please, in what form do you prefer: nick (burtek), first name (Bartosz) or will you provide the full name?

@burtek
Copy link
Contributor Author

burtek commented May 15, 2024

either works :) I usually default to my username when coding, so let's go with that

Thanks!

@Serg-Norseman
Copy link
Owner

Ok :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants