BaRatin-tools / BaRatinAGE

BaRatin Advanced Graphical Environment
https://baratin-tools.github.io/en/
GNU General Public License v3.0
4 stars 0 forks source link

A more flexible approach to build a hydraulic configuration #13

Open IvanHeriver opened 1 year ago

IvanHeriver commented 1 year ago

I noticed that duplicating a hydraulic configuration is not very useful when you change the number of controls afterwards (which is a common thing to do I think) since all the parameter values are reset...

I would expect the following behavior:

@benRenard and @JeromeLeCoz, what do you think?

Enhancement to consider for v3?

JeromeLeCoz commented 1 year ago

Fully agreed. I think we had considered this enhancement for future versions. However, as you suggest, it'd require some options for managing the controls we keep or not, which may not be trivial to implement.

Ideally, I'd like to be able to assign existing controls to new controls (i.e. select what controls must be kept and re-order them).

For instance: if I duplicate a 2 control config and ask for 3 controls, I'd like to say that now C2 is former C1 and C3 is former C2. If I duplicate a 3 control config and ask for 2 controls, I'd like to say that now C1 is former C1 and C2 is former C3 (and former C2 is no longer used).

Perhaps some kind of drag and drop system coule be used for this, a bit like in BaMit! to select the inputs and parameters?

benRenard commented 1 year ago

It's a recurrent request and indeed what is needed is the ability to copy-paste or drag-and-drop controls. But it might also suggest a modification of the "hydraulic configuration" panel: instead of starting with the matrix we could start with individual controls, and then build the matrix by dragging-and-dropping individual controls (which could hence easily be reused in several configurations). It's not obvious to do and it's kind of a big change so it needs some thinking. Could be a v3 thing indeed.

IvanHeriver commented 10 months ago

The initial problem mentioned at the start of the issue is now fixed in BaRatinAGE v3.

However, the more flexible approach to build a hydraulic configuration using drag and drop or existing controls from other configuration is still not address. I'll rename the issue accordingly.