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.

Sign up for GitHub

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

Jump to bottom

Complex Particle Model + Breakups #103

Merged
merged 83 commits into from
Feb 15, 2022
Merged

Complex Particle Model + Breakups #103

merged 83 commits into from
Feb 15, 2022

Conversation

FG-TUM
Copy link
Collaborator

@FG-TUM FG-TUM commented Feb 1, 2022 •
edited
Loading

Description

  • Restore integration test -> needs Quickly deorbiting Particles #100 to be resolved first
  • Resolve TODOs in default_cfg.yaml
  • Link to propagator version that uses table based atmospheric density

Related Pull Requests

Combines:

Resolved Issues

All the issues resolved in the PRs above

How Has This Been Tested?

  • 1 month simulation compared against SGP4

@FG-TUM
remove @brief
50a90f0
@FG-TUM
extend Particle by ActivityState:
5f423c0 
 - enum in Particle class
 - update all constructor calls
@FG-TUM
update propagator to temporary commit that includes change for accele…
13dc91f 
…rator
@FG-TUM
formatting and missing new constructor arguments in tests
138c03f
@FG-TUM
mass and radius for particles
ccab314
@FG-TUM
only register conjunctions if they involve two passive or one passive…
371ce7d 
… small particle
@FG-TUM
fix mistake in exclusion logic and add missing cases to ActivityState…
0bec337 
… test
@FG-TUM
update propagator and switch it to new fork
e4c01b0
@FG-TUM
move HDF5 definitions from writer to dedicated file
14d537a
@FG-TUM
update propagator (minor variable rename)
ba026ea
@FG-TUM
proper equality operator for Particle
b660fa7
@FG-TUM
read and write static particle data to HDF5 + tests
2758077
@FG-TUM
extend Particle constructor by mass and radius
817bf71
@FG-TUM
don't try to write when there is nothing to write
923e7b5
@FG-TUM
constellations are evasivePreserving by default
030b1d5
@FG-TUM
formatting
598c186
@FG-TUM
only generate a csv from the propagator when explicitly specified in …
c1f56df 
…yaml
@FG-TUM
first implementation of burn up
ad04d8e
@FG-TUM
remove unnecessary alias
c67bdb5
@FG-TUM
refactor existing SimulationTest to reduce code duplication and impro…
82bec41 
…ve readability
@FG-TUM
burn up test
651994a
@FG-TUM
move converter tests to new directory ot mirror src
2f656ac
@FG-TUM
update repo path to breakup code
65c4a2d
@FG-TUM
SimulationTest: restructure config parameters in only needed here and…
4ce40c2 
… necessary
@FG-TUM
update AutoPas to get AutoPas::deleteParticle(particle &).
06b4ae8 
This now points to a branch. Will need update as soon as it is merged.
@FG-TUM
Activate breakup Model:
585a4d7 
  - BreakupWrapper to wrap all interaction with the external code
  - new input parameter
  - new timer to measure time spent in breakups
  - new test
@FG-TUM
doc
9c62f00
@FG-TUM
use enforceMassConservation from the yaml instead of a const value
a7c43b2
@FG-TUM
change > to != which is the same here
a6da895
@FG-TUM
doc
693dbe1
@FG-TUM FG-TUM force-pushed the mergeBranch branch 3 times, most recently from e8cb84a to cc1e4bc Compare February 14, 2022 19:10
@FG-TUM
reactivated validation tests with new reference
6ebde55 
    - made name of expected output file a input var
    - validation script also gives positive feedback
    - add git-lfs to CI
@FG-TUM
[WIP] Fix validation tests (#113)
8b32177 
reactivated validation tests with new reference

    - made name of expected output file a input var
    - validation script also gives positive feedback
    - add git-lfs to CI
@FG-TUM FG-TUM requested a review from gomezzz February 15, 2022 01:19
gomezzz
gomezzz requested changes Feb 15, 2022
View reviewed changes
README.md Show resolved Hide resolved
cfg/default_cfg.yaml Outdated Show resolved Hide resolved
cfg/default_cfg.yaml Show resolved Hide resolved
cfg/testValidation.yaml Outdated Show resolved Hide resolved
tests/data/README.md Show resolved Hide resolved
tests/validationtest.py Show resolved Hide resolved
@FG-TUM FG-TUM mentioned this pull request Feb 15, 2022
Merged
6 tasks
@FG-TUM FG-TUM requested a review from gomezzz February 15, 2022 18:57
@FG-TUM FG-TUM merged commit 08fce77 into main Feb 15, 2022
@FG-TUM FG-TUM deleted the mergeBranch branch February 15, 2022 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gomezzz gomezzz gomezzz approved these changes

Assignees

@FG-TUM FG-TUM

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@FG-TUM @gomezzz