Repository navigation
HighRiseTPU: split each segment's load with its height within the story - #436
Open
concrete-sangminlee wants to merge 1 commit into
Open
concrete-sangminlee wants to merge 1 commit into
concrete-sangminlee wants to merge 1 commit into
Conversation
addForcesFace() used locY, the absolute height of the segment, to split its force between the floor above and the floor below, so for every story above the first Rabove > A and Rbelow < 0, and most of the mean load landed on the roof. Use the height within story i, as LowRiseTPU.cpp does since 5db635e; locY stays absolute for findNearestTAP(). Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #435
addForcesFace()inmodules/createEVENT/HighRiseTPU/HighRiseTPU.cppsplit each mesh segment's force between the floor above and the floor below withlocY, the segment's absolute height, instead of its height within the current story. For every story above the first this givesRabove > AandRbelow < 0; the total is conserved, but most of the mean load ends up on the roof (about 99 % for TPU model T111_4 at 0°, see #435).This change uses the height within story
ifor the split and keepslocY(absolute) forfindNearestTAP(). It is algebraically the same correction thatLowRiseTPU.cppreceived in 5db635e (lines 685–688 there).Checked on a build of 12b6f99 with this hunk (the tested build differs from this branch only in the wording of the comment):
HighRiseTPUbinary differs between the two installed trees, andweuq-0001gives a byte-identicaldakotaTab.out;T111_4_000_1.mat), 26 stories: every floor is loaded, the roof gets about half a typical floor, and the total mean load is unchanged (11,439 kN);h·L·qon each floor andh·L·q/2on the roof, as expected;🤖 Generated with Claude Code