Skip to content
This repository has been archived by the owner on Jun 21, 2024. It is now read-only.

Fix inspection for Makie 0.18 #116

Merged
merged 2 commits into from
Nov 16, 2022
Merged

Conversation

fbanning
Copy link
Member

Fixes issues with DataInspector from #113 which raised Makie compat to v0.18. This PR should cover all changes made to DataInspector. It now looks a lot nicer as well. :)

Screenshot_20221115_180155
Screenshot_20221115_180203

NB: This PR builds on top of the changes made in #114 and currently it's necessary to ]add OSMMakie#master to test this PR. It can only be merged once OSMMakie v0.0.6 is available on the public registry (it's registered and will be auto-merged with the next batch). To indicate this, I will leave this as a draft PR and only convert it to ready once the registry is updated.

@fbanning fbanning added bug Something isn't working Agents Concerning plotting of Agents.jl models labels Nov 15, 2022
@fbanning fbanning self-assigned this Nov 15, 2022
@fbanning
Copy link
Member Author

Ah nevermind, OSMMakie v0.0.6 is already published! 🎉 So this PR is good to go as well. :)

You OK with this, @Datseris ?

@fbanning fbanning marked this pull request as ready for review November 15, 2022 17:13
@fbanning fbanning merged commit 9132312 into graphspace-support Nov 16, 2022
@fbanning fbanning deleted the inspection-makie-0.18 branch November 16, 2022 16:57
@Datseris
Copy link
Member

Datseris commented Dec 1, 2022

Did you publish a new tag of InteractiveDynamics with this in? (patch number should have been incremented here, as no v1 exists yet)

@fbanning
Copy link
Member Author

fbanning commented Dec 2, 2022

No I didn't because I merged it into the graphspace-support branch which already has an incremented version number. These should be published together.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Agents Concerning plotting of Agents.jl models bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants