Skip to content
This repository has been archived by the owner on Oct 23, 2022. It is now read-only.

Adding Pluto and Some sample notebooks #80

Merged
merged 5 commits into from
Apr 7, 2021
Merged

Conversation

galran
Copy link
Collaborator

@galran galran commented Apr 6, 2021

This PR focus on adding Pluto and the some Pluto notebooks to the package.

added dependency on new packages: Pluto, PlutoUI, WGLMakie, Format
Note: WGLMakie can clash with GLMAkie - we need to maybe create an init function to resolve it once and for all.
Added the NotebooksUtils module to help repeating notebook code.
Added 2 utility functions to the Vis module to allow better control of rendering inside Pluto notebooks.
added 3 sample notebooks in the samples/notebooks folder and a utility function to launch Pluto and run them.
added a small description in the documentation Examples page instructing how to launch the notebooks.
added a call to the GLMakie.activate!() function in the documentation script to make sure the GLMakie backend is used.
Bug Fix: Seems like i forgot to include the new emitters page in the make.jl script that produce the documentation pages.

Note: WGLMakie can clash with GLMAkie - we need to maybe create an init function to resolve it once and for all.
Added the NotebooksUtils module to help repeating notebook code.
Added 2 utility functions to the Vis module to allow better control of rendering inside Pluto notebooks.
added 3 sample notebooks in the samples/notebooks folder and a utility function to launch Pluto and run them.
added a small description in the documentation Examples page instructing how to launch the notebooks.
added a call to the GLMakie.activate!() function in the documentation script to make sure the GLMakie backend is used.
Bug Fix: Seems like i forgot to include the new emitters page in the make.jl script that produce the documentation pages.
@codecov-io
Copy link

codecov-io commented Apr 6, 2021

Codecov Report

Merging #80 (ec98a2c) into main (ff8b93d) will decrease coverage by 1.84%.
The diff coverage is 2.05%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #80      +/-   ##
==========================================
- Coverage   48.64%   46.79%   -1.85%     
==========================================
  Files          56       56              
  Lines        6474     6569      +95     
==========================================
- Hits         3149     3074      -75     
- Misses       3325     3495     +170     
Impacted Files Coverage Δ
src/NotebooksUtils/NotebooksUtils.jl 0.00% <0.00%> (ø)
src/Visualization/Visualization.jl 6.03% <0.00%> (+0.07%) ⬆️
src/OpticSim.jl 80.00% <75.00%> (-3.34%) ⬇️
src/Geometry/Primitives/Qtype.jl 75.00% <0.00%> (-8.34%) ⬇️
src/Optical/LensAssembly.jl 48.22% <0.00%> (-7.33%) ⬇️
src/Optical/OpticalInterface.jl 62.68% <0.00%> (-4.42%) ⬇️
src/Geometry/CSG/Intersection.jl 48.57% <0.00%> (-2.14%) ⬇️
src/Geometry/Geometry.jl 0.00% <0.00%> (-1.86%) ⬇️
src/Geometry/CSG/Interval.jl 83.20% <0.00%> (-0.37%) ⬇️
src/Geometry/Primitives/Curves/BSpline.jl 77.51% <0.00%> (-0.35%) ⬇️
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ff8b93d...ec98a2c. Read the comment docs.

@galran
Copy link
Collaborator Author

galran commented Apr 6, 2021

@alfredclwong @BrianGun
seems like the ubuntu runs are stuck. this PR introduced the WGLMakie into the mix and maybe it's confusing the rendering part again :(

Ran Gal and others added 4 commits April 6, 2021 22:07
@BrianGun BrianGun merged commit d68caf3 into main Apr 7, 2021
@BrianGun BrianGun deleted the galran/new_samples branch April 7, 2021 20:51
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.

3 participants