Skip to content

Conversation

ChrisRackauckas-Claude
Copy link

Summary

  • Fixed initialization warnings by providing complete initial conditions for all state variables
  • Added documentation note about Pluto-specific variable naming requirements
  • Updated sensitivity analysis code to use unique variable names

Changes

  1. Complete initial conditions: Added initial values for all state variables (L1.i, inertia.w, inertia.phi, pi_controller.int.x) to avoid underdetermined system warning
  2. Pluto compatibility note: Added a note explaining that Pluto requires unique variable names and how to resolve "Multiple definitions" errors
  3. Unique variable names: Renamed simplified_sys variables in sensitivity analysis to avoid conflicts (simplified_sys_S, simplified_sys_T, simplified_sys_L)

Test plan

  • Created test script to verify changes
  • Confirmed complete initial conditions eliminate underdetermined warning
  • Verified sensitivity analysis runs without variable name conflicts
  • Tested that incomplete initial conditions still produce expected warning

Fixes #410

🤖 Generated with Claude Code

ChrisRackauckas and others added 2 commits September 19, 2025 08:55
- Add complete initial conditions for all state variables (L1.i, inertia.w, inertia.phi, pi_controller.int.x) to avoid underdetermined system warning
- Add note about Pluto-specific variable naming requirements
- Use unique variable names in sensitivity analysis to avoid conflicts

Fixes SciML#410

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas merged commit 797dd31 into SciML:main Oct 5, 2025
3 of 4 checks passed
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.

DC Motor Example warnings in Pluto

2 participants