Skip to content
Snippets Groups Projects

pymodulepackage (schueltke): fix for tool_level access

Merged Florian Schültke requested to merge fix/tool_level_mission into develop
All threads resolved!

Problem:

  • when tool_level in aircraft exchange file is accessed by a module but is not existing, the pymodulepackage throws an error as it can be seen and discussed in weight_and_balance_analysis: aircraft-design!31 (comment 2743760) Solution:
  • implement a handling procedure how to continue, when tool_level access failed and set tool_level to 0 in that case

Test:

  • test previous version using weight_and_balance_analysis with an aircraft exchange file in a first loop, when tool_level is not yet written in the node masses_cg_inertia and compare it with an execution when the attribute was already written by weight_and_balance_analysis
  • test the same procedure with the fix from this MR

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
  • Florian Schültke requested changes

    requested changes

  • Florian Schültke resolved all threads

    resolved all threads

  • Florian Schültke aborted the automatic merge because the source branch was updated. Learn more.

    aborted the automatic merge because the source branch was updated. Learn more.

  • added 1 commit

    • 4452d12d - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Kristina Mazur added 3 commits

    added 3 commits

    Compare with previous version

  • requested review from @AndiGob

  • Andi approved this merge request

    approved this merge request

  • Andi enabled an automatic merge when all merge checks for 65d1500a pass

    enabled an automatic merge when all merge checks for 65d1500a pass

  • Andi bypassed reviews on this merge request

    bypassed reviews on this merge request

  • Andi canceled the automatic merge

    canceled the automatic merge

  • merged

  • Andi mentioned in commit e999a694

    mentioned in commit e999a694

  • Please register or sign in to reply
    Loading