Skip to content
Snippets Groups Projects

performance_assessment report + removed fuelPlanning

Merged performance_assessment report + removed fuelPlanning
fix/performance_assesment_OS into develop
Merged Philipp Hansmann requested to merge fix/performance_assesment_OS into develop

Description

  • Added performance_assessment report
  • removed switches for submodules from config and code
  • removed fuelPlanning and missionOptimization (relies on fuel planning)
  • Repository has been forked by @philipp.hansmann before removin fuelPlanning and missionOptimization
  • Performend major cleanup; not quite thorough though

Related Issue(s)

  • Closes #[feature issues] There was an issue for that but i dunno right now

Other Changes

Screenshots/Logs

Testing Instructions

  1. [Step 1: How to test]
  2. [Step 2: Expected outcome]
  3. [Step 3: Additional steps if required]

Developer Checklist

  • Code has been tested locally and/or in pipeline.
  • (if applicable) documentation updated.
  • (if applicable) impact of new dependencies reviewed and included in project.
  • Merge conflicts resolved with the target branch.

Additional Notes

Add any information reviewers should focus on, e.g., specific files, functions, or changes of interest.

Edited by Philipp Hansmann

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Philipp Hansmann added 1 commit

    added 1 commit

    • ea39bee1 - deleted old node from config

    Compare with previous version

  • Philipp Hansmann resolved all threads

    resolved all threads

  • Philipp Hansmann added 1 commit

    added 1 commit

    • 45079ff6 - deactivated execution of engine_performance, SAR_performance, BFL, Vn +...

    Compare with previous version

  • Philipp Hansmann added 1 commit

    added 1 commit

    • 1c642a6a - fixed minor error in html report

    Compare with previous version

  • Philipp Hansmann added 1 commit

    added 1 commit

    • 3eb4d666 - fix for v1 being written in acxml even when not calculated

    Compare with previous version

  • Florian Schültke @Florian.Schueltke started a thread on an old version of the diff
    • performance_assessment/performance_assessment_conf.xml
    51 56 <performance_checks description="Settings that affect different functionalities!">
    52 <payload_range_diagram description="Payload Range Diagram Calculation">
    53 <switch description="On-Off-Switch for this method: 1:On, 0:Off" Unit="-" Default="1">1</switch>
    54 <fidelity_level description="Fidelity level of calculation">low</fidelity_level>
    57 <payload_range_diagram description="Payload Range Diagram calculation">
    58 <fidelity_level description="Select the fidelity level of the calculation (options: low)">
    59 <value>low</value>
    60 </fidelity_level>
    55 61 </payload_range_diagram>
    56 62 <engine_performance description="Engine power estimation">
    57 <switch description="On-Off-Switch for this method: 1:On, 0:Off" Unit="-" Default="1">1</switch>
    58 <fidelity_level description="Fidelity level of calculation">low</fidelity_level>
    63 <fidelity_level description="Select the fidelity level of the calculation (options: low)">
    64 <value>low</value>
    65 </fidelity_level>
    59 66 </engine_performance>
  • Florian Schültke added 1 commit

    added 1 commit

    • e259496b - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Florian Schültke added 1 commit

    added 1 commit

    • 09f47f1f - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Florian Schültke requested review from @kristina.mazur

    requested review from @kristina.mazur

  • F
    Florian Schültke @Florian.Schueltke ·
    Owner

    I cannot approve anymore since I made two small changes in whitespaces. I tested it and can approve these changes

  • F
    Florian Schültke @Florian.Schueltke ·
    Owner

    @kristina.mazur please help us with this dilemma :smile:

  • Kristina Mazur approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
    • Please register or sign in to reply