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

Add docs section about custom plots #80

Merged
merged 4 commits into from
Dec 7, 2021
Merged

Add docs section about custom plots #80

merged 4 commits into from
Dec 7, 2021

Conversation

fbanning
Copy link
Member

@fbanning fbanning commented Dec 2, 2021

This docs section is added in response to JuliaDynamics/Agents.jl#551.

@Datseris I would like to add two screenshots to this section. Where would you normally store those? I saw that you've stored the videos for the docs landing page on the JuliaDynamics github space.

@Datseris
Copy link
Member

Datseris commented Dec 2, 2021

Hi @fbanning cool! Yeap we store "external" images here https://github.com/JuliaDynamics/JuliaDynamics/tree/master/videos/agents in order to minimize download size!

@fbanning fbanning marked this pull request as ready for review December 3, 2021 13:08
@fbanning
Copy link
Member Author

fbanning commented Dec 3, 2021

I don't need to bump the patch number for a docs update, right?

@Datseris
Copy link
Member

Datseris commented Dec 3, 2021

for the change to be reflected in the stable docs yes. However at the moment master branch is unstable because graph space uses Graphs.jl and OSM uses LightGraphs.jl. So don't bother with bversions!

@fbanning
Copy link
Member Author

fbanning commented Dec 7, 2021

at the moment master branch is unstable

You might be thinking about Agents right now. As far as I can tell, InteractiveDynamics does not have anything specifically unstable in its current master branch. If you're OK with my additions in this PR, I would say you could already merge it.

@Datseris Datseris merged commit ae37339 into JuliaDynamics:master Dec 7, 2021
@Datseris
Copy link
Member

Datseris commented Dec 7, 2021

It is getting really hard to keep track of many large projects.... :D

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants