Skip to content

Skycultures 2.0: Arabic Lunar Mansions#4494

Merged
gzotti merged 4 commits intomasterfrom
SC/arabLM
Sep 8, 2025
Merged

Skycultures 2.0: Arabic Lunar Mansions#4494
gzotti merged 4 commits intomasterfrom
SC/arabLM

Conversation

@gzotti
Copy link
Member

@gzotti gzotti commented Sep 4, 2025

Description

The Arabic lunar mansions needed some extra definitions to allow variable extent (width) of each mansion. This is now used only in the "arabic_indigenous" SC.
Khalid further refined the definitions of the Arabic skycultures.

Fixes # (issue)

Screenshots (if appropriate):

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • This change requires a documentation update
  • Housekeeping

How Has This Been Tested?

Test Configuration:

  • Operating system: Windows 11
  • Graphics Card: (irrelevant)

Checklist:

  • My code follows the code style of this project.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (header file)
  • I have updated the respective chapter in the Stellarium User Guide
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@gzotti gzotti added this to the 25.3 milestone Sep 4, 2025
@gzotti gzotti self-assigned this Sep 4, 2025
@gzotti gzotti added the feature Entirely new feature label Sep 4, 2025
@github-actions
Copy link

github-actions bot commented Sep 4, 2025

Great PR! Please pay attention to the following items before merging:

Files matching src/**/*.cpp:

  • Are possibly unused includes removed?

Files matching skycultures/**:

  • Did you remember to update skycultures/CMakeLists.txt file respectively to changes in sky cultures?
  • Did you remember to define classification parameter in sky cultures (see index.json file)?
  • Did you remember to define license parameter in sky cultures (see description.md file)?
  • Did you remember to define region parameter in sky culture (see index.json file)?

This is an automatically generated QA checklist based on modified files.

@github-actions
Copy link

github-actions bot commented Sep 4, 2025

Hello @gzotti!

Thank you for proposing of the feature.

Copy link
Member

@alex-w alex-w left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@gzotti
Copy link
Member Author

gzotti commented Sep 5, 2025

Don't merge yet please. I want to fix the long labels for zodiac/lunar, allow selection of one name part only.

- now pronounce contains "simple" Latin spelling, adaptable to user language preferences.
- transliteration contains the more correct but non-typewriter diacritics
- Arabic Lunar Mansions removed. No longer a separate SC - integrated into the others
@gzotti gzotti merged commit d4c3381 into master Sep 8, 2025
32 checks passed
@gzotti gzotti deleted the SC/arabLM branch September 8, 2025 10:45
@github-project-automation github-project-automation bot moved this from Backlog to Done in Skycultures 2.0 Sep 8, 2025
@alex-w alex-w added the state: published The fix has been published for testing in weekly binary package label Sep 17, 2025
@github-actions
Copy link

Hello @gzotti!

Please check the fresh version (development snapshot) of Stellarium:
https://github.com/Stellarium/stellarium-data/releases/tag/weekly-snapshot

@alex-w alex-w removed the state: published The fix has been published for testing in weekly binary package label Sep 29, 2025
@github-actions
Copy link

Hello @gzotti!

Please check the latest stable version of Stellarium:
https://github.com/Stellarium/stellarium/releases/latest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Entirely new feature

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants