-
Notifications
You must be signed in to change notification settings - Fork 7
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
Model Repository Parser for UML Diagrams #144
Conversation
Thanks for update, @ismonnar! There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2024-06-13 11:22:27 UTC |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #144 +/- ##
==========================================
- Coverage 56.17% 48.72% -7.46%
==========================================
Files 102 112 +10
Lines 9025 10957 +1932
==========================================
+ Hits 5070 5339 +269
- Misses 3955 5618 +1663
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
добавил тебя в разрабы репоза, чтобы не делать ПР из форка
сделай плиз чекаут от мэйна и оформи ПР уже из ветки индастриала
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
всё супер, лишь один момент поправить и можно вливать
uml/uml.py
Outdated
with open('../fedot_ind/core/repository/model_repository.py', 'r') as file: | ||
file_content = [string for string in file.read().split("\n") if not string.startswith("from")] | ||
file_content = file_content[:next((i for i, line in enumerate(file_content) if line.startswith("def")), None)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
я кажется упоминал, что лучше привязываться к project_path из path_lib
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
изменил файл
вы ещё просили добавить его в директорию tools - тоже сделал
Парсер репозитория моделей для создания UML-диаграмм