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

Introducing f3d::options #191

Merged
merged 6 commits into from Jan 13, 2022
Merged

Conversation

mwestphal
Copy link
Contributor

@mwestphal mwestphal commented Jan 8, 2022

In the context of #52 , Introduce a first element, f3d::options and start using it when possible without refactoring the whole of F3D for now.

TODO in other PRs:

src/library/f3dOptions.h Outdated Show resolved Hide resolved
src/library/F3DOptions.cxx Outdated Show resolved Hide resolved
src/library/F3DLoader.cxx Outdated Show resolved Hide resolved
src/library/F3DLoader.cxx Show resolved Hide resolved
src/library/vtkF3DGenericImporter.h Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Jan 10, 2022

Codecov Report

Merging #191 (3d91a60) into master (9fc91ff) will increase coverage by 0.19%.
The diff coverage is 99.56%.

❗ Current head 3d91a60 differs from pull request most recent head b808465. Consider uploading reports for the commit b808465 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #191      +/-   ##
==========================================
+ Coverage   88.26%   88.46%   +0.19%     
==========================================
  Files          71       74       +3     
  Lines        3733     3840     +107     
==========================================
+ Hits         3295     3397     +102     
- Misses        438      443       +5     
Impacted Files Coverage Δ
src/library/F3DAnimationManager.h 100.00% <ø> (ø)
src/library/F3DOptions.h 100.00% <ø> (ø)
src/library/vtkF3DGenericImporter.h 100.00% <ø> (ø)
src/library/vtkF3DRenderer.h 50.00% <ø> (ø)
src/library/vtkF3DRendererWithColoring.h 58.82% <ø> (ø)
src/library/vtkF3DRenderer.cxx 95.16% <96.87%> (+0.03%) ⬆️
src/library/F3DAnimationManager.cxx 93.75% <100.00%> (+0.06%) ⬆️
src/library/F3DLoader.cxx 95.34% <100.00%> (+0.19%) ⬆️
src/library/F3DOperators.h 100.00% <100.00%> (ø)
src/library/F3DOptions.cxx 98.00% <100.00%> (+0.37%) ⬆️
... and 9 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 9fc91ff...b808465. Read the comment docs.

@mwestphal mwestphal changed the title Libf3d api proto Introducing f3d::options Jan 10, 2022
@mwestphal
Copy link
Contributor Author

@Meakk : please review

@mwestphal mwestphal mentioned this pull request Jan 10, 2022
32 tasks
src/library/F3DOperators.h Outdated Show resolved Hide resolved
src/library/f3d_options.cxx Outdated Show resolved Hide resolved
@mwestphal mwestphal merged commit ac7cbd9 into f3d-app:master Jan 13, 2022
@mwestphal mwestphal deleted the libf3d_API_proto branch July 10, 2022 17:32
mwestphalnew pushed a commit to mwestphalnew/f3d that referenced this pull request Feb 10, 2024
Add STEP and IGES file support

Closes f3d-app#191

See merge request f3d/f3d!236
mwestphalnew pushed a commit to mwestphalnew/f3d that referenced this pull request Feb 10, 2024
In the context of f3d-app#52 , Introduce a first element, f3d::options and start using it when possible without refactoring the whole of F3D for now.

TODO in other PRs:
- doc
- F3DLoader refacto
- options names f3d-app#32
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.

None yet

2 participants