pyiron / pyiron_workflow / 12605227009
92%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 33
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 91.505% (+0.004%) from 91.501%
12605227009

push

github

web-flow 
Draft: Ruff (#520)

* add ruff linter/formatter to CI

* show diff

* enable common ruff rules

* exclude some files from formatting

* sort imports

* apply ruff auto-fixes

* apply unsafe fixes -- NEEDS REVIEW!

* ignore line-length violations as they are covered by the formatter

* reset _version and add it to excludes

* fix logger reimport

* fix over-eager optimizations in tests

* format fixes

* some custom fixes

* deactivate F401 for main include

* Get rid of creator and wrapper caches

Signed-off-by: liamhuber <liamhuber@greyhavensolutions.com>

* De-abstract HasCreator

* Remove unused import

Signed-off-by: liamhuber <liamhuber@greyhavensolutions.com>

* Ignore unused imports in init files\n\nwe specify APIs this way

* Remove ABC

I personally like to use it as an indicator to devs that there's no intention of actually instantiating this class itself, but ruff insists that they are _only_ for specifying interfaces, and without and abstractmethod this doesn't make the cut. I acquiesce.

Signed-off-by: liamhuber <liamhuber@greyhavensolutions.com>

* Don't use lru_cache

The operation is simple enough that the savings must have been minimal, and this is not used repeatedly (only when parsing class definitions to start with, e.g. at initial import), so any minor cost is going to be tolerable anyhow.

Signed-off-by: liamhuber <liamhuber@greyhavensolutions.com>

* Use a more descriptive variable

Signed-off-by: liamhuber <liamhuber@greyhavensolutions.com>

* Use more descriptive variables

Signed-off-by: liamhuber <liamhuber@greyhavensolutions.com>

* Use more specific settings field

Per ruff deprecation warning

Signed-off-by: liamhuber <liamhuber@greyhavensolutions.com>

* Keep the Workflow import separate

It was placed intentionally separate from the other imports to distinguish the pyironic single-import user interface from the more standard user/dev public API imports.... (continued)

3070 of 3355 relevant lines covered (91.51%)

0.92 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
1
98.39
 0.0% nodes/function.py
1
97.73
 0.0% type_hinting.py
1
93.64
 0.0% mixin/run.py
2
98.2
 0.0% nodes/for_loop.py
2
96.23
 0.04% storage.py
3
98.25
 0.01% nodes/transform.py
3
85.71
 -0.29% nodes/static_io.py
4
92.0
 -1.44% create.py
5
58.44
 0.0% executors/cloudpickleprocesspool.py
6
96.67
 -0.02% mixin/semantics.py
7
92.14
 0.0% nodes/macro.py
7
94.76
 -0.02% channels.py
9
91.67
 0.42% nodes/composite.py
10
94.24
 -0.07% io.py
12
91.38
 0.42% nodes/standard.py
14
90.91
 -0.03% node.py
21
66.17
 0.0% workflow.py
Jobs
ID Job ID Ran Files Coverage
1 12605227009.1 33
91.51
 GitHub Action Run
Source Files on build 12605227009