-
Notifications
You must be signed in to change notification settings - Fork 157
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
feat: GeometryView3D can write to specified outputDir #917
Conversation
Codecov Report
@@ Coverage Diff @@
## main #917 +/- ##
==========================================
- Coverage 48.75% 48.71% -0.04%
==========================================
Files 331 331
Lines 17103 17119 +16
Branches 8070 8084 +14
==========================================
+ Hits 8339 8340 +1
- Misses 3057 3061 +4
- Partials 5707 5718 +11
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trivial change.
In #917 I introduced an output directory to `GeometryView3D`. I set the default to "." as that usually means the current directory. Due to the way we currently look for file extensions, this didn't actually work. I didn't actually test the default, so that broke the output writing. This PR works around this by checking if the default value "." is given, and replaces this with the actual `$PWD`. This is not super robust, and ideally we should revise all of this using `boost::filesystem`, which I'll try to do in a future PR. Aside from this, this PR also make the geometry examples correctly pipe through the `--output-dir` option to the geometry view output.
Previously, they would always write to
$PWD
which is not desirable.