Skip to content

Account for array parameters in IndexCache #4273

Draft
jClugstor wants to merge 1 commit intoSciML:masterfrom
jClugstor:array_affect
Draft

Account for array parameters in IndexCache #4273
jClugstor wants to merge 1 commit intoSciML:masterfrom
jClugstor:array_affect

Conversation

@jClugstor
Copy link
Member

Checklist

  • Appropriate tests were added
  • Any code changes were done in a way that does not break public API
  • All documentation related to code changes were updated
  • The new code follows the
    contributor guidelines, in particular the SciML Style Guide and
    COLPRAC.
  • Any new documentation only uses public API

Additional context

Add any other context about the problem here.

@jClugstor
Copy link
Member Author

Attempt to address #4272

@jClugstor
Copy link
Member Author

@AayushSabharwal the issue in #4272 was that tunable_idxs had entries like c[1], c[2], but arrsym is just c. So trying to do tunable_idxs[arrsym] = idxs failed. This fixes that particular issue but I'm not sure if it's sensible to do it this way or if it breaks anything else.

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.

1 participant