Merge main into loopdemo

[liamhuber]

@juliusmittler, the logs launching binder right on julius_loop_demo complained about configuration of the hatching build system's versioning tool. When I peek at this branch (and your working branch), I see it still mixes-and-matches the old setuptools build system and the new hatchling build system. I patched all these back in January 27 on main, but I guess that's just not merged into your branch.

Here, I checked a new branch off julius_loop_demo and merged main. There were conflicts for the .gitignore, the pptx in the docs, and all the notebooks (I bet they've just been re-executed). I accepted main uniformly here. This merge didn't hurt any of your work and we get all your nice changes to the underlying source code and the demo is showing up fine. I can launch it directly from this binder link and it looks like it's working to me:

https://mybinder.org/v2/gh/pyiron/pyironflow/merge-main-into-loopdemo?urlpath=%2Fdoc%2Ftree%2Floop_demo.ipynb

If this all looks good and works well for you, I expect it's also trivially straightforward and helpful to merge main into your main julius_gui dev branch. As long as the conflicts there are also all documentation-related, it should be similarly safe to accept the main version of each conflict file. You might want to be more intelligent about merging .gitignore in case you added meaningful things to it on your branch.

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[liamhuber]

notebook failure:

File [/srv/conda/envs/notebook/lib/python3.13/site-packages/pyironflow/reactflow.py:336](https://hub.gesis.mybinder.org/srv/conda/envs/notebook/lib/python3.13/site-packages/pyironflow/reactflow.py#line=335), in PyironFlowWidget.update(self)
    334 def update(self):
--> 336     nodes = get_nodes(self.wf, self.gui.expanded_macros)
    337     edges = get_edges(self.wf, self.gui.expanded_macros)
    339     self.gui.nodes = json.dumps(nodes)

TypeError: get_nodes() missing 1 required positional argument: 'buildExpand'

I went to look at the signature for get_nodes and I see you modified it with an extra parameter. I haven't looked into the how and why, but I have no intrinsic objection to that. This is the sort of test failure we need to get resolved before merging your branch into main, but not before merging main into your branch -- as long as the demo notebook is behaving as you want, I think it's ok to merge this into your demo branch.