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

Add interactive HQMTool and testing #15

Merged
merged 165 commits into from
Jun 24, 2022
Merged

Add interactive HQMTool and testing #15

merged 165 commits into from
Jun 24, 2022

Conversation

DavidAKopriva
Copy link
Collaborator

Add testing of HQMTool. No coverage on interactive project notifications or on plotting.

DavidAKopriva and others added 20 commits September 9, 2021 08:38
Add the code, documentation and examples to the repo for the HQMTool interactive front end.
* Integrate docs

* Store png on GitHub

* Push documentation previews

* Fix docs; move sources

* Move AllFeatures.control to examples folder

* Add Printf dependency

* Rearrange files

* Fix path

* Fix docs

* Hopefully make tests pass again
Start adding unit tests. First Batch is for curves. Need to figure out how to organize tests into multiple files and then one to rule them all.
Add tests for background grid, run parameters, smoother.
Add tests for refinement regions. fix exposed bugs.
Add more refinement tests and associated bug fixes.
Cover all but two (plotting point) methods.
Finish up background grid tests, and fix exposed bugs.
Finish up smoother tests.
Finish up coverage.
Add coverage and undo/redo.
All but remove inner curve tested
Update to add model tests to main test file.
Do tests on project, except for notification actions.
Add project tests to suite
@codecov
Copy link

codecov bot commented Mar 24, 2022

Codecov Report

Merging #15 (dde3e22) into main (8a781fc) will decrease coverage by 1.14%.
The diff coverage is 98.83%.

@@             Coverage Diff             @@
##              main      #15      +/-   ##
===========================================
- Coverage   100.00%   98.85%   -1.15%     
===========================================
  Files            1       20      +19     
  Lines           40     1838    +1798     
===========================================
+ Hits            40     1817    +1777     
- Misses           0       21      +21     
Flag Coverage Δ
unittests 98.85% <98.83%> (-1.15%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/ControlFile/ControlFileOperations.jl 91.66% <91.66%> (ø)
src/Project/Project.jl 97.52% <97.52%> (ø)
src/Curves/Spline.jl 98.38% <98.38%> (ø)
src/Project/CurvesAPI.jl 98.42% <98.42%> (ø)
src/Project/ModelAPI.jl 99.06% <99.06%> (ø)
src/Curves/CurveOperations.jl 100.00% <100.00%> (ø)
src/HOHQMesh.jl 100.00% <100.00%> (ø)
src/Mesh/Meshing.jl 100.00% <100.00%> (ø)
src/Misc/DictionaryOperations.jl 100.00% <100.00%> (ø)
src/Misc/NotificationCenter.jl 100.00% <100.00%> (ø)
... and 11 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 de8c375...dde3e22. Read the comment docs.

src/HOHQMesh.jl Outdated Show resolved Hide resolved
src/HOHQMesh.jl Outdated Show resolved Hide resolved
* add authors section to docs

* add contributing section to docs

* add development section to the docs

* typo fixes

* update main docs page. missing links

* ignore auto-generated authors file

* separate HQMTool docs for better clarity. Add to CheatSheet.md

* fix typo in filename

* change header names in HQMTool docs

* seperate workflow and HQMTool tour in docs

* fix docstrings in background grid API

* typo fix in workflow

* section out the tour

* point to the tour and tutorials

* remove broken reference links

* change name of section

* update description in the tour

* add information about HQMTool to the front page of docs

* fix broken image link

* small readibility edits

* clarify the API

* rearrange API docs section

* point to Trixi docs for mesh format explanation

* update comments in the tour

* add link to tutorials introduction

* fix typo in spline information

* add tutorials section

* add curved blob boundary tutorial

* typo fixes in new docs files

* typo fixes in interactive example scripts

* add straight-sided two circles tutorial

* add spline curves tutorial

* clarify output of new splines tutorial

* add description of the undo redo tutorial

* update tutorial link in API docs

* comment out unfinished doc file

* docs should build now

* typos fixes

* state explicitly Julia compatibility

* more typo fixes

* first draft of editing curves tutorial

* typos fixes and add synopsis of each tutorial to the main page

* typo fixes in edit curve tutorial

* typo fixes

* add synopsis also to the top of tutorials

* update section name of tutorials introduction

* quality of life improvement where adding curves display a one line message

* adjust wording to not call the project variable a dictionary

* clean up exporting to only be front facing functions. Tests still pass

* adjust tests with new exporting strategy

* adjust comments in Undo and Meshing

* change tutorial file name

* edit docs for clarity

* adjust file names in the docs
@andrewwinters5000 andrewwinters5000 merged commit e26aa9c into main Jun 24, 2022
@andrewwinters5000 andrewwinters5000 deleted the ToolTesting branch June 24, 2022 09:33
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.

5 participants