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 ExportCoordinates executable #1417

Merged
merged 1 commit into from
Mar 15, 2019

Conversation

nilsvu
Copy link
Member

@nilsvu nilsvu commented Mar 5, 2019

Proposed changes

This executable is useful to export the domain coordinates so that we can interpolate externally computed initial data to it.

Once the Kerr executable is evolving (#1358) the idea is to compute initial data for a perturbed Kerr black hole with SpEC, interpolate it to our domain and try to evolve that. To do that we can export our domain coordinates with this executable, use the SpEC tools to interpolate to these coordinates, and then import the interpolated data. I will provide Python code to do the interpolation with SpEC and the importing functionality in a future PR.

Types of changes:

  • New feature

Component:

  • Code

Code review checklist

  • The PR passes all checks, including unit tests, clang-tidy and IWYU.
    For instructions on how to perform the CI checks locally refer to the Dev
    guide on the Travis CI
    .
  • The code is documented and the documentation renders correctly. Run
    make doc to generate the documentation locally into BUILD_DIR/docs/html.
    Then open index.html.
  • The code follows the stylistic and code quality guidelines listed in the
    code review guide.

Copy link
Contributor

@mar-celine mar-celine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only some minor high level comments from me 👍

Also, it may be beyond the scope of this PR, but it would be nice to be able to visualize a domain in paraview without running a simulation using that domain. It was something I was considering figuring out how to do myself, but if this executable is not too far from being able to accomplish that, then that is something I would appreciate having 😄

@nilsvu
Copy link
Member Author

nilsvu commented Mar 8, 2019

@gsb76 indeed you can use this executable to output the grid of a domain and then run any of our visualisation tools over it. (That's an overstatement, there's really only GenerateXdmf.py in the repository so far. But using that I made this picture in like 1 minute)

bildschirmfoto 2019-03-08 um 10 00 03

@nilsvu
Copy link
Member Author

nilsvu commented Mar 11, 2019

@kidder could you review this or find a reviewer?

@kidder kidder self-requested a review March 11, 2019 23:02
@kidder
Copy link
Member

kidder commented Mar 11, 2019

Yes, I can review it

Copy link
Member

@kidder kidder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also the tests are failing because of a missing option...

template <size_t Dim>
struct Metavariables {
static constexpr OptionString help{
"Export the domain coordinates. The output can be used to compute "
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Export the inertial coordinates of the Domain specified in the input file. ...

#include "Utilities/TMPL.hpp"

namespace {
// We only need this type for the `ObservationId`. Can be removed with #1408.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#1408 was just merged

@nilsvu nilsvu force-pushed the spec_initial_data branch 2 times, most recently from 5434660 to 436e89e Compare March 12, 2019 09:12
@nilsvu
Copy link
Member Author

nilsvu commented Mar 12, 2019

@kidder addressed your comments, please check the fixups

@kidder
Copy link
Member

kidder commented Mar 12, 2019

okay, please squash

This executable is useful to export the domain coordinates so that
we can interpolate externally computed initial data to it.
@nilsvu
Copy link
Member Author

nilsvu commented Mar 13, 2019

done @kidder

@nilsvu
Copy link
Member Author

nilsvu commented Mar 13, 2019

@gsb76 could you sign off on this when you get the chance?

@kidder kidder merged commit 38dbdc0 into sxs-collaboration:develop Mar 15, 2019
@nilsvu nilsvu deleted the spec_initial_data branch October 2, 2019 15:00
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.

3 participants