Skip to content

Conversation

jurasic-pf
Copy link
Collaborator

No description provided.

Copy link
Collaborator Author

jurasic-pf commented Apr 9, 2025


How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@jurasic-pf jurasic-pf requested a review from eguiraud-pf April 9, 2025 11:28
@jurasic-pf jurasic-pf marked this pull request as ready for review April 9, 2025 11:28
@jurasic-pf jurasic-pf removed the request for review from eguiraud-pf April 9, 2025 11:28
@jurasic-pf jurasic-pf marked this pull request as draft April 9, 2025 11:29
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

const Eigen::VectorXi& xm_nyq,
const Eigen::VectorXi& xn_nyq,
RowMatrixXd& currumnc, RowMatrixXd& currvmnc) {
const int ns = bsubsmns.rows();
Copy link

Choose a reason for hiding this comment

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

warning: narrowing conversion from 'Index' (aka 'long') to signed type 'int' is implementation-defined [bugprone-narrowing-conversions]

  const int ns = bsubsmns.rows();
                 ^

@jurasic-pf jurasic-pf force-pushed the jurasic-wout-compatibility branch from dd2289d to 99a2d11 Compare April 9, 2025 11:52
@jurasic-pf jurasic-pf force-pushed the jurasic-wout-currumnc branch from 322257d to 38fb975 Compare April 9, 2025 11:52
@jurasic-pf jurasic-pf changed the title Add Wout currumnc, currvmnc (untested) [WOut] Add currumnc, currvmnc (untested) Apr 9, 2025
@jurasic-pf jurasic-pf force-pushed the jurasic-wout-compatibility branch from 99a2d11 to d434009 Compare April 9, 2025 12:15
@jurasic-pf jurasic-pf force-pushed the jurasic-wout-currumnc branch from 38fb975 to 0e257b7 Compare April 9, 2025 12:15
@jurasic-pf jurasic-pf changed the base branch from jurasic-wout-compatibility to graphite-base/235 April 13, 2025 22:17
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.

Some wout attributes are present in C++ but not exposed from the Python layer
1 participant