Skip to content

feat: improve default names#349

Merged
mortenengen merged 7 commits into
devfrom
improve-default-names
Jun 10, 2026
Merged

feat: improve default names#349
mortenengen merged 7 commits into
devfrom
improve-default-names

Conversation

@mortenengen

@mortenengen mortenengen commented Apr 21, 2026

Copy link
Copy Markdown
Member

Description

This is an improvement to the naming of core components in the library. It is an attempt to implement a global counter for materials, constitutive laws, geometries and sections in a consistent manner, and use these counters as means for creating default object names when names are not given at object initialization.

To do

  • Implement base_name argument in Geometry base class.
  • Remove group_label from Geometry base class and move to only those classes where it is relevant.

@mortenengen mortenengen added the enhancement New feature or request label Apr 21, 2026
@mortenengen mortenengen moved this to Under review 👀 in PR tracker Apr 21, 2026
@mortenengen mortenengen marked this pull request as ready for review April 21, 2026 08:48

@talledodiego talledodiego left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this PR. I have some minor questions that probably are due to me missing something.

Comment thread structuralcodes/core/base/_geometry.py
Comment thread structuralcodes/core/base/_constitutive_law.py
Comment thread structuralcodes/core/base/_constitutive_law.py
@mortenengen mortenengen moved this from Under review 👀 to Changes requested 📝 in PR tracker May 28, 2026

@talledodiego talledodiego left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

excellent design. I like the idea of the mixin

@mortenengen mortenengen merged commit 811d181 into dev Jun 10, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from Changes requested 📝 to Done 🚀 in PR tracker Jun 10, 2026
@mortenengen mortenengen deleted the improve-default-names branch June 10, 2026 06:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done 🚀

Development

Successfully merging this pull request may close these issues.

2 participants