Skip to content

Conversation

@FG-TUM
Copy link

@FG-TUM FG-TUM commented Jan 25, 2022

Description

Adds a minimal example on how to include this project in a C++ code.

  • minimal CMakeLists.txt
  • minimal main()

@gomezzz gomezzz requested a review from darioizzo January 25, 2022 16:45
int main() {
// generate satellites to collide
SatelliteBuilder satelliteBuilder;
std::vector<Satellite> satellites{
Copy link
Member

Choose a reason for hiding this comment

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

So I guess the satellite type is not here as it does not impact the results? If it does, whats the default value? Maybe you can set it too here?

Copy link
Author

@FG-TUM FG-TUM Jan 26, 2022

Choose a reason for hiding this comment

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

The type impacts the results. From the documentation:

Needed for determining the right equations for breakup.
If not given this will be always a SPACECRAFT.

I left it out since I only wanted to give a minimal example here and there are plenty of examples in the example-config folder for the demo code in main.cpp.

std::vector<Satellite> satellites{
satelliteBuilder
.setID(0)
.setPosition({0.,0.,0.}) // [m]
Copy link
Member

Choose a reason for hiding this comment

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

WHat happens to this position information? Discarded? Or it gets checked that the two sats are close enough to collide?

Copy link
Author

Choose a reason for hiding this comment

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

afaik only the position of the first satellite matters as this is used as the point where the debris is spawned.

@FG-TUM FG-TUM requested a review from darioizzo January 26, 2022 17:19
@FG-TUM FG-TUM requested a review from gomezzz January 27, 2022 12:39
Copy link
Collaborator

@gomezzz gomezzz left a comment

Choose a reason for hiding this comment

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

For me fine. What about you @darioizzo ?

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