pwptemp icon indicating copy to clipboard operation
pwptemp copied to clipboard

Casings

Open elyngvi opened this issue 4 years ago • 2 comments

Added a to-do task. Today the model accepts an arbitrary 3 casings, and it is not straight forward to define them, as you have to give the shoe depths one place and the dimensions somewhere else.

I suggest you take away all the 9 (?) casing related properties, and replace them with one, "casings" which takes an array of any number of casings, defined by depth and diameters: 'casings': [{'od': num, 'id': num, 'md': num}, ...] This array could be sorted by md for example, to make sure the casings are in the right order. You would have to rewrite your code a bit to fit that change, but it would be a great benefit.

elyngvi avatar Oct 21 '19 08:10 elyngvi

Excellent suggestion! Hopefully this will be available for the next release.

jcamiloangarita avatar Nov 01 '19 20:11 jcamiloangarita

All the parameters are available into the same dictionary and the casings names have been changed for a better understanding.

Additionally, a function pwptemp.input.info( ) was created to get information regarding all the parameters.

jcamiloangarita avatar Nov 06 '19 19:11 jcamiloangarita