Skip to content

fix: justification fix, beam offset calc fixes, ifc and glb export fixes and more#182

Open
oleandor wants to merge 25 commits intoKrande:mainfrom
oleandor:fix/justification_and_more
Open

fix: justification fix, beam offset calc fixes, ifc and glb export fixes and more#182
oleandor wants to merge 25 commits intoKrande:mainfrom
oleandor:fix/justification_and_more

Conversation

@oleandor
Copy link
Contributor

@oleandor oleandor commented Mar 4, 2026

No description provided.

@oleandor
Copy link
Contributor Author

oleandor commented Mar 5, 2026

@Krande, please have a look when you find the time. test_write_step seems to fail, and i sort of understand why, not sure what is the best way to handle it. we could set the section made by outer_poly to general section, and then try to calc the Cgy, Cgz ect, - or we can just bypass it in curve_offset_local. not sure how much this outer_poly defiend section is used

@Krande
Copy link
Owner

Krande commented Mar 11, 2026

Hey, sorry for slow response. Thanks for this PR! Will try to take a look this week or the next:)

@oleandor
Copy link
Contributor Author

oleandor commented Mar 16, 2026

I think I’ve solved it now. The COG calculation should now be consistent with the ADA model definition, GXML export/import, FEM import, beam justifications, visualizations in the ADA viewer, as well as IFC and GLB exports.

I’ve also updated the Para model based on this PR, so hopefully you’re happy with the changes and we can release the Para model with this dependency update. 🙂

@github-actions
Copy link

👋 Hi there! I have checked your PR and found no issues. Thanks for your contribution!

PR Review:

I found no pr-related issues.

  • ✅ PR title is ok
  • ✅ Release label is ok
  • ✅ SOURCE_KEY is set as a secret
  • ✅ Skipping release

Python Review:

I found no python-related issues.

Python Linting results:

  • ✅ Isort
  • ✅ Black
  • ✅ Ruff

Python Packaging results:

  • ✅ I found the PYPI_API_TOKEN secret.
Packaging Type Package Name Version
pyproject.toml ada-py 0.7.11
pypi ada-py 0.7.11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants