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

[FEATURE]: Show bays in diagram overview #122

Closed
wants to merge 15 commits into from
Closed

Conversation

interBN
Copy link
Contributor

@interBN interBN commented Oct 14, 2024

@interBN
Copy link
Contributor Author

interBN commented Nov 5, 2024

Findings:

export function extractIEDInfosWithBay(root: Element): Map<string, IEDCommInfo[]> {
if (!root) {
console.info({ level: "info", msg: "initInfos: no root" })
return new Map()
}
const scdQueries = new SCDQueries(root)
const ucci = new UCCommunicationInformation(scdQueries)
const iedInfosByBay = ucci.IEDCommInfosByBay()
return iedInfosByBay
}

Put the var root in this function to get extract bays.

The bays should be loaded somewhere here.

@interBN interBN force-pushed the feat/show-bays-in-ce branch 2 times, most recently from 3135fdf to 952d158 Compare November 12, 2024 09:00
This was unlinked from issues Nov 13, 2024
@interBN interBN linked an issue Dec 3, 2024 that may be closed by this pull request
2 tasks
@interBN
Copy link
Contributor Author

interBN commented Dec 3, 2024

This branch is based on the branch feat/show-bays-in-diagram. Therefore we need to wait until feat/show-bays-in-diagram in ready to merged in main.

@interBN
Copy link
Contributor Author

interBN commented Dec 3, 2024

see: #221 (comment)

@pwinkelholzS1 pwinkelholzS1 linked an issue Dec 18, 2024 that may be closed by this pull request
3 tasks
This was unlinked from issues Dec 18, 2024
@pwinkelholzS1
Copy link
Collaborator

will be resolved with Feat/select bays in sidebar #253

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

Successfully merging this pull request may close these issues.

2 participants