Use instance check instead of equality in as_dataclass_node

[pmrv]

type(cls) is only equal to type if no other meta class has been used. This includes classes derived from ABC, so it is a bit inconvenient. Using the instance check however seems to work in these cases.

[github-actions]

👈 Launch a binder notebook on branch pyiron/pyiron_workflow/dataclass_typecheck

[codacy-production]

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
✅ +0.03% (target: -1.00%)	✅ 100.00%

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (bd13cc3)	3422	3126	91.35%
Head commit (e79c196)	3432 (+10)	3136 (+10)	91.38% (+0.03%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#601)	1	1	100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

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

[coveralls]

Pull Request Test Coverage Report for Build 13540109426

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 6 unchanged lines in 1 file lost coverage.
• Overall coverage increased (+0.03%) to 91.375%

---

Files with Coverage Reduction	New Missed Lines	%
mixin/run.py	6	94.17%

Totals
Change from base Build 13509927617:	0.03%
Covered Lines:	3136
Relevant Lines:	3432

---

💛 - Coveralls

[liamhuber]

Yes, good catch. Now downsides are obvious to me, and the upside is clear. Once the CI passes, I'm happy.

[pmrv]

Seems codacy is slow today. Based on its last known output I was gonna ignore it anyway as we discussed on zoom, so I'll go ahead an merge.