[minor] Move stuff

[liamhuber]

This is a refactor, but since there was not a concrete API and things are in different places, it's a minor rather than patch bump.

I took everything downstream of Node (except Workflow) and moved it into the submodule nodes, and everything upstream of Node (or mixed into a subclass) and Channel and moved it into the submodule mixin. IMO this makes the entire project structure easier to understand.

While users are still intended to use Workflow as a single-point-of entry -- and supported in this by the creator/wrapper menus -- I also found that people developing nodes seem to like to import stuff like as_function_node directly and use it. This is now explicitly supported by a "node developer API" section of the __init__ file which has all the standard factories, decorators, creators, and parent classes that should be necessary for day-to-day creation of new node packages.

[review-notebook-app[bot]]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[github-actions[bot]]

:point_left: Launch a binder notebook on branch _pyiron/pyiron_workflow/movestuff

[codacy-production[bot]]

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
:white_check_mark: +0.05% (target: -1.00%)	:white_check_mark: 95.24%

Coverage variation details

| | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (50656b2057a3f090424575cc7cb8b5a150d4bc09) | 3418 | 3163 | 92.54% | | | Head commit (30e2ced0b90cffbcc05780fe6402c4c9ea3a5b81) | 3428 (+10) | 3174 (+11) | 92.59% (**+0.05%**) | **Coverage variation** is the difference between the coverage for the head and common ancestor commits of the pull request branch: ` - `

Diff coverage details

| | Coverable lines | Covered lines | Diff coverage | | ------------- | ------------- | ------------- | ------------- | | Pull request (#362) | 126 | 120 | **95.24%** | **Diff coverage** is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: `/ * 100%`

See your quality gate settings    Change summary preferences

---

_{:rocket: Don’t miss a bit, follow what’s new on Codacy.}

_{Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more}

[liamhuber]

I considered and (for now) rejected the idea of moving things the nodes/channels use but don't inherit from into their own submodule (e.g. draw, parse_output, topology, etc, into tools or similar). I found unlike the mixins/nodes which could be clearly identified by their up/downstream position in the Node inheritance hierarchy, for these objects there was no such clear-cut way to distinguish what should be moved into the submodule and what should be left at the main level.

It is still clear that something like topology is more niche than something like io, so I see room to further pare down the main level of the module in the future -- I just don't have a clear vision for how to do it right now.

[coveralls]

Pull Request Test Coverage Report for Build 9470936620

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 28 unchanged lines in 2 files lost coverage.
• Overall coverage increased (+0.05%) to 92.59%

---

Files with Coverage Reduction	New Missed Lines	%
io.py	12	94.35%
workflow.py	16	80.87%
<!--	Total:	28		-->

Totals
Change from base Build 9457867783:	0.05%
Covered Lines:	3174
Relevant Lines:	3428

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 9470958205

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 30 unchanged lines in 2 files lost coverage.
• Overall coverage increased (+0.05%) to 92.59%

---

Files with Coverage Reduction	New Missed Lines	%
io.py	14	94.35%
workflow.py	16	80.87%
<!--	Total:	30		-->

Totals
Change from base Build 9457867783:	0.05%
Covered Lines:	3174
Relevant Lines:	3428

---

💛 - Coveralls