Skip to content

Conversation

@pepbos
Copy link
Owner

@pepbos pepbos commented May 6, 2024

Draft PR to discuss during meeting

cable. */
enum class Status
{
Ok,
Copy link
Owner Author

Choose a reason for hiding this comment

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

WrappingStatus::InContactWithSurface

/** Compute the unit force vector in Ground frame that this segment exerts
on the MobilizedBody. The actual applied force can be found by multiplication with the cable tension.
The system must be realiezd to Stage::Position. */
void calcUnitForce(const State& state, SpatialVec& unitForce_G) const;
Copy link
Owner Author

Choose a reason for hiding this comment

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

This can be removed

that if nPoints=1, and the curve length is not zero, an exception is
thrown.
@return The number of points written. */
int calcPoints(const State& state, std::vector<Vec3>& points_G, int nPoints = 0) const;
Copy link
Owner Author

Choose a reason for hiding this comment

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

use a std::function<void(Vec3)>& sink

or use an epsilon? use will be not be happy choosing nPoints...

belong to a valid cable path.*/
CurveSegment(
CableSpan cable,
const MobilizedBody& mobod,
Copy link
Owner Author

Choose a reason for hiding this comment

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

Take an index instead


//==============================================================================
// A curved cable segment on a surface that is part of a `CableSpan`.
class SimTK_SIMBODY_EXPORT CurveSegment final
Copy link
Owner Author

Choose a reason for hiding this comment

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

should this be exposed at all?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants