STARIONGROUP / COMET-IME-Community-Edition

The Concurrent Design Desktop Application and Excel Integration compliant with ECSS-E-TM-10-25 Annex A and Annex C
https://www.stariongroup.eu
Other
21 stars 5 forks source link

Auto-refresh parameter values in Parameter to State Mapper #1240

Closed stepjah closed 6 months ago

stepjah commented 1 year ago

Prerequisites

Description

The Parameter to State Mapper does not consider updated ParameterValues when loading an existing mapping but reads the value that is stored in the target mapping string. Only after manually deselecting and selecting the mapped state again, the new parameter value is read from the model and updated in the mapping table. This is almost impossible to maintain for big models and continuous changes of the parameter values. The user need is to once define the mapping to a state and then always dynamically update the mapped value. An easy solution would be to implement a "refresh" button in the Parameter to State Mapper once going through all defined mappings and updating the mapped values in the mapping table. Alternatively the mapped value should not be stored in the target mapping string at all and the parameter to state mapper always directly reading from the model when loading a mapping.

Steps to Reproduce

Logs

System Configuration

stepjah commented 7 months ago

Is there a chance to put some priority on this issue and implement this feature in an upcoming version? This is a pain point that is reported frequently from the projects applying the tool. Thanks a lot!

lxatstariongroup commented 6 months ago

Is there a chance to put some priority on this issue and implement this feature in an upcoming version? This is a pain point that is reported frequently from the projects applying the tool. Thanks a lot!

@stepjah: You can find the new IME 9.7.0 release candidate here: https://github.com/RHEAGROUP/COMET-IME-Community-Edition/releases/tag/9.7.0-rc1 Please let me know if this works for you.

stepjah commented 6 months ago

@lxatrhea I can confirm that the update solves this issue and is a great improvement!