Skip to content

Conversation

@ommishra0719
Copy link

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

I added a one-line dispatch to defsdict in src/systems/system.jl to handle Vector{Any}:
defsdict(d::Vector{Any}) = isempty(d) ? Dict{Any, Any}() : defsdict(Dict(d))

@hersle
Copy link
Contributor

hersle commented Jan 26, 2026

Would fix #4206.

@ommishra0719
Copy link
Author

ommishra0719 commented Jan 27, 2026

Do I need to make any other changes, since it didn't pass a few tests?

@AayushSabharwal
Copy link
Member

Not necessarily. Some of these failures look spurious. I reran the necessary jobs to make sure.

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.

3 participants