Conversation
…kp tuning
New references/commands/primitive_wall.md: when to prefer primitive_wall
over mesh, and a pointer to its own docs for restrictions (no motion
mechanism) and output behavior (no VTK output) -- both since documented
directly in primitive_wall.html's own Restrictions section.
STRATEGIES.md: three new entries -- verifying a per-particle-state command
actually took effect (mark_particles/define_group/enable_heat_transfer can
run with no error while doing nothing), tilting gravity during a
non-physical prep phase to pack particles at a target location for free,
and mesh_module servo's kp often needing to be much larger than its
default to actually reach maximum_velocity.
RULES.md: one-line addition cross-referencing the verification entry above
under Simulation Output.
variable.md: merged the existing boolean-only \${name}-nesting warning with
a newly confirmed equal-formula case (same underlying bug, extends to
equal/atom/boolean formulas alike) into one section instead of two.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…veat RULES.md: two new entries under Command Ordering/Simulation Output -- only declare a physics setting (e.g. heating) in the script phase that actually needs it, since a packing/prep phase split off via write_restart/read_restart can silently inherit one left over from an earlier script version; and give each named script its own output_settings folder, clearing it (not just the restart file) before rerunning outside a full clean, since Aspherix renumbers rather than overwrites colliding step files with no error. STRATEGIES.md: two new entries -- mesh_module servo's center_of_mass must track the pushed body's actual current position when the same piston geometry is reused across scripts/restarts, since the mesh's imported STL coordinates don't update on their own; and only regenerate a restart when the phase that produced it actually needs to change, since re-running an earlier phase unnecessarily wastes wall-clock time on an expensive DEM/CFD case. SKILL.md: link primitive_wall.md from the Guidelines list (it existed but wasn't linked). Co-Authored-By: Claude Sonnet 5 <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
1 participant
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.
No description provided.