Skip to content

Conversation

@engpas
Copy link
Contributor

@engpas engpas commented Feb 27, 2025

Description

Update schemas, resources, examples and tests to reflect new parameter handling in jms.

See #553

Checklist

  • I have tested these changes locally.
  • I have added unit tests (if appropriate).
  • I have added necessary documentation or updated existing documentation.
  • I have linked the issue(s) addressed by this PR if any.

@codecov
Copy link

codecov bot commented Feb 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.56%. Comparing base (109eb3f) to head (3a61ecf).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #547   +/-   ##
=======================================
  Coverage   87.56%   87.56%           
=======================================
  Files          65       65           
  Lines        2894     2894           
=======================================
  Hits         2534     2534           
  Misses        360      360           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@engpas engpas marked this pull request as ready for review February 27, 2025 12:51
Copy link
Contributor

@ojkoenig ojkoenig left a comment

Choose a reason for hiding this comment

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

Looked through it, might have found one error in mapdl_motorbike_frame example, see comment

@engpas engpas requested a review from ojkoenig March 3, 2025 09:17
@engpas engpas linked an issue Mar 12, 2025 that may be closed by this pull request
4 tasks
Copy link
Contributor

@ojkoenig ojkoenig left a comment

Choose a reason for hiding this comment

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

Please coordinate with Federico when it can be merged

@engpas engpas changed the title Pengeler/laki reqs parameters Added explicit mode specification Mar 27, 2025
@engpas engpas changed the title Added explicit mode specification Add explicit mode specification for parameters Mar 27, 2025
@engpas engpas merged commit 8065326 into main Mar 27, 2025
22 checks passed
@engpas engpas deleted the pengeler/laki-reqs-parameters branch March 27, 2025 08:01
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.

Explicit parameter mode specification

4 participants