Skip to content

Conversation

@vyudu
Copy link
Contributor

@vyudu vyudu commented Aug 13, 2025

fix #38

@MasonProtter
Copy link
Collaborator

We'll also want a method in ext/MTKExt.jl that converts Nums to Symbols, i.e.

function SymbolicIndexingInterface.observed(sys::PartitionedGraphSystem, vars::Union{Vector{Num}, Tuple{Vararg{Num}}})
    SymbolicIndexingInterface.observed(sys, tosymbol.(vars; escape=false))
end

@vyudu
Copy link
Contributor Author

vyudu commented Aug 27, 2025

@MasonProtter is this waiting on something?

@MasonProtter
Copy link
Collaborator

Oops, im sorry I forgot about this! Could you just change the test so that it gets run by CI?

It's currently just in a free-floating file i think

@vyudu
Copy link
Contributor Author

vyudu commented Sep 5, 2025

fixedup, should be mergeable

@MasonProtter MasonProtter merged commit 2ab9554 into Neuroblox:master Sep 5, 2025
7 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.

Support vectors of Observed variables in the SymbolicIndexingInterface

2 participants