Skip to content

[Atscale-26967] catalogs packages perspectives updates #28

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged

Conversation

stanislavtrifonov88
Copy link
Contributor

@stanislavtrifonov88 stanislavtrifonov88 commented May 30, 2025

  • update catalogs and perspectives documentation

We had to do a research on perspectives for another task and noticed an issue in the documentation. It incorrectly states that the list of dimensions includes only those to be included, however, it should actually list all dimensions, hierarchies, and levels that need to be excluded.

Copy link
Collaborator

@diannewood diannewood left a comment

Choose a reason for hiding this comment

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

Perspectives are a particularly complex and confusing part of SML. I think we should add more detail to this section.

….com:semanticdatalayer/SML into ATSCALE-26967-catalogs-packages-perspectives
Copy link
Collaborator

@diannewood diannewood left a comment

Choose a reason for hiding this comment

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

One more comment to address

Copy link
Collaborator

@diannewood diannewood left a comment

Choose a reason for hiding this comment

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

LGTM!

@stanislavtrifonov88 stanislavtrifonov88 merged commit 8f4ced8 into main May 30, 2025
1 check passed
@github-actions github-actions bot locked and limited conversation to collaborators May 30, 2025
@svetoslavpetkov svetoslavpetkov deleted the ATSCALE-26967-catalogs-packages-perspectives branch July 22, 2025 09:30
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants