-
Notifications
You must be signed in to change notification settings - Fork 10
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
Split up core in smaller sub modules #821
Conversation
…pression broke) actually this is very strange, as the relative import triggers a behaviour to use relative import all over the place. The weldx.util.util pkg imports an attr from asdf, but this import would be tried to resolve by Python (3.10) to the weldx internal "asdf" package.
Codecov Report
@@ Coverage Diff @@
## master #821 +/- ##
==========================================
+ Coverage 96.80% 97.25% +0.44%
==========================================
Files 81 82 +1
Lines 5380 4913 -467
==========================================
- Hits 5208 4778 -430
+ Misses 172 135 -37
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
In my opinion the code is better maintainable, if modules are kept smaller. The complains of static analysis are not new, but now show up as being newly introduced. |
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.
I agree, this should be easier to maintain and keep clean 👍
thanks for doing this
Changes
Split up core in smaller sub modules, e.g. generic_series and mathematical_expression and so forth.