Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature/graphical fixes #1119

Merged
merged 4 commits into from
Mar 17, 2025
Merged

feature/graphical fixes #1119

merged 4 commits into from
Mar 17, 2025

Conversation

rhayes777
Copy link
Owner

Resolves #1110

@Jammy2211
Copy link
Collaborator

I ran the following integration test:

https://github.com/Jammy2211/autolens_workspace_test/blob/main/slam_graph/integration/base.py

For the most part, this works, however the model.graph file is now output in files as a .json file, which is fine but it was also a text file before which gave the users information, so keeping it also as a text file next to model.info would be good:

image

Although maybe just keeping it as model_graph.json is fine, but at the moment the format is very human unreadable:

image

@Jammy2211
Copy link
Collaborator

Nevertheless I will merge this as itg'll help me test graphical models.

@Jammy2211 Jammy2211 merged commit 4245d8b into main Mar 17, 2025
0 of 4 checks passed
@Jammy2211 Jammy2211 deleted the feature/graphical_fixes branch March 24, 2025 19:41
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.

Graphical Model 1: model.info and model.graph
2 participants