Skip to content

RevitMechanicalSpecificationHelpersUpdate: Вспомогательные процессы - #489

Open
Redomine wants to merge 9 commits into
masterfrom
Redomine/RevitMechanicalSpecificationHelpersUpdate
Open

RevitMechanicalSpecificationHelpersUpdate: Вспомогательные процессы #489
Redomine wants to merge 9 commits into
masterfrom
Redomine/RevitMechanicalSpecificationHelpersUpdate

Conversation

@Redomine

Copy link
Copy Markdown
Contributor
  1. Проверки занятости вынесены из ParamChecker в ElementEditChecker, занятые элементы теперь пропускаются.
    2)Изменён пайплайн MaskReplacer: результаты имени и марки передаются через SpecificationElement. Это необходимо для обработки вложений - параметры ADSK у таких элементов блокируются, не позволяя использование масок.
  2. Добавлено автоматическое пересоздание повернутых горизонтальных воздуховодов с сохранением подключений и изоляции.

Отредактированы сценарии при которых занятый элемент попадал в обработку
…соре, чтобы избежать ситуаций когда семейство вложенное и его параметры ADSK_ блокируются
… повернутость и пересоздание в случае не прохождения.
@Redomine
Redomine requested a review from dosymep August 21, 2026 13:31
@Redomine Redomine self-assigned this Aug 21, 2026

@dosymep dosymep left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • обнови форматирование кода, у тебя нет использования нового C#

Comment thread src/RevitMechanicalSpecification/Entities/ElementParameterValue.cs Outdated
}
}

public void TryApply(Parameter parameter) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

я бы добавил проверку соответствия StorageType у параметра и сохраненного значения

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

чтобы во время исключения было понятно какая ошибка и где, а то тут будет InvalidCastException без контекста

…раметров воздуховод сразу отмечается как проблемный и пропускается.

2) Первый из проблемных воздуховодов теперь запоминается по айди, которое пишется в текст ошибки. Их в целом на модель должно быть несколько штук, поэтому этого должно хватить для первоначальной диагностики.
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