RevitMechanicalSpecification: Создан плагин #86
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…араметров воздуховодных сетей
…яра в отдельный статический класс, перенес конвертер на статический класс, добавил обработчик масок имени-марки
…ов которая возвращала значения до встречи отводов
dosymep
requested changes
Sep 20, 2024
There was a problem hiding this comment.
- рекомендую каждый метод команды ревита вынести в разные классы из RevitRepository, не надо в одном классе все смешивать, это приемлимо если класс выходит не большой
- перепроверь все названия методов, ты где-то называешь
Paramгде-тоParameterнадо быть последовательным в именовании
src/RevitMechanicalSpecification/Models/Fillers/ElementParamFiller.cs
Outdated
Show resolved
Hide resolved
src/RevitMechanicalSpecification/Models/Fillers/ElementParamFiller.cs
Outdated
Show resolved
Hide resolved
src/RevitMechanicalSpecification/Models/Fillers/ElementParamFiller.cs
Outdated
Show resolved
Hide resolved
src/RevitMechanicalSpecification/Models/Fillers/ElementParamGroupFiller.cs
Outdated
Show resolved
Hide resolved
src/RevitMechanicalSpecification/Models/Fillers/ElementParamGroupFiller.cs
Show resolved
Hide resolved
Убраны лишние поля
…нтов делим на отдельностоящие и внутри узлов
…етры проекта. Поставил просто GetParam
Пока временно пришлось закастомить добавление параметров.
…ринудительного имени системы и принудительной функции изоляции из воздуховодов
…за нее отвечает старый параметр ФОП_ВИС_Экономическая, а за остальные принудиловки отвечают именно параметры _принудительное. Может просто использовать глобальные параметры?
Redomine
changed the title
dosymep
approved these changes
Oct 2, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Создан плагин для формирования девятиграфной формы спецификации в механических разделах
Возможности:
В дальнейшем планируется расширение функционала и добавление графического интерфейса настроек.