-
Notifications
You must be signed in to change notification settings - Fork 21
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
more mypy checks, docopt-->argparse, formatting, cleanup, ci improvements, ... #405
Conversation
06cfee9
to
b8696db
Compare
f8e776b
to
60f5ad1
Compare
2a46def
to
54cb5d5
Compare
54cb5d5
to
8f1acce
Compare
I think the settings need some change to accomodate for the removed "CI Tests Successful" job. However I do not have sufficient access rights for that. I've changed the settings accordingly. "CI Tests Successful" is no longer necessary. Instead, "Execute CI Tests" is now considered. Best |
c5d7b29
to
2a03053
Compare
2a03053
to
d185b6d
Compare
|
…e remains stable though) UNTESTED!
43b4bd6
to
3bf0a4a
Compare
3bf0a4a
to
99809b8
Compare
unless you see any issues, I think this is ready for merging. @lukasrothenberger |
if cast(Workload, node_data).parallelizable_workload is None | ||
else cast(int, cast(Workload, node_data).parallelizable_workload) | ||
) | ||
min_wl = (node_data.sequential_workload or 0) + (node_data.parallelizable_workload or 0) |
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.
nice solution! :)
No description provided.