hominoids / SBC_Case_Builder

Autonomous SBC case generation using SBC Model Framework
GNU General Public License v3.0
370 stars 23 forks source link

Cleanup Case Builder Config #5

Closed davejlong closed 1 year ago

davejlong commented 1 year ago

It seems like there's a number of elements of the Case Builder config file that aren't used or are potentially the wrong data type. I figured I'd create a list of them to make sure they aren't there for some future plan before submitting a PR to remove or change them:

hominoids commented 1 year ago

The standoff reverse and standoff insert can be boolean but keep the standoff height for now. It doesn't need to be in the customizer but keep it in the case schema(sbc_case_builder.cfg). I have been working on decoupling the standoff height so it's not a calculation, in order to provide additional case flexibility. On one hand keeping it as a calculation removes the user from having to figure it out and possibly make a mistake, but it also doesn't allow some case configurations. I may be able to do the case config's with variable standoff height in the code base, and if so, it can be taken out later

davejlong commented 1 year ago

What if, instead of a standoff height, it's a modification to the calculated standoff? For example, if you want to add additional buffer space, perhaps to make room for a fan, you would just set the height to be the fan's size, and that value is added to the calculation?

hominoids commented 1 year ago

Extra z-axis is achieved by case_offset_bz already. I think we will be going in the direction chosen by the decisions made regarding the customizer, see issue#7.