JuliaHEP / JuliaHEP-2023

Materials for the JuliaHEP 2023 Workshop
https://juliahep.github.io/JuliaHEP-2023/
Creative Commons Attribution 4.0 International
4 stars 4 forks source link

Plots tutorial [now with extra Makie bonus content] #20

Closed aoanla closed 10 months ago

aoanla commented 10 months ago

Hello all,

This is the current version of the plots tutorial, which goes a little fast [and could probably be even faster]. I added 1 Makie and 1 AlgebraOfGraphics example at the end just so people can see them being used for something, but there's no actually tutorialising in those parts.

I also made 1 edit to the Broadcasting tutorial to mention that broadcasting over a row and a column vector does the outer product, as I use it here and also in another case to make a quick matrix.

aoanla commented 10 months ago

As far as I can tell, the deploy-book is failing because it runs as "me", and I don't have edit perms on the base repo here.

graeme-a-stewart commented 10 months ago

As far as I can tell, the deploy-book is failing because it runs as "me", and I don't have edit perms on the base repo here.

I just gave you maintain rights...

aoanla commented 10 months ago

Hm, I have no idea why this is still failing (for you and me) - "Resource not accessible by integration" usually means that there's a permissions issue somewhere.... but this is on the deploy part that should be trivial (it's not when the book is actually built)

aoanla commented 10 months ago

Aha, I see the problem, the "deploy preview" action is https://github.com/rossjrw/pr-preview-action which explicitly notes "This Action does not currently support deploying previews for PRs from forks, but will do so in https://github.com/rossjrw/pr-preview-action/pull/6." Since this is a PR from a fork...

Now I have maintain rights here, though, I can probably push this branch into a branch of this main repo, and then make a pull request for that (which should work with this action).

Moelf commented 10 months ago

well don't worry you can probably just merge this and we will see