TALA icon indicating copy to clipboard operation
TALA copied to clipboard

Published 20 hours ago • verified publisher icon terrastruct

Tala improvement suggestion: reduce empty space

Open judepayne opened this issue 2 years ago • 5 comments

Hi, On large diagrams, Tala introduces quite a lot of empty space. Would it be possible to introduce a setting to 'compress' the space?

image

judepayne avatar Jan 27 '23 09:01 judepayne

most recent TALA release has an improvement related to this. can you try rerunning? (or post the script if it's still bad) @judepayne

alixander avatar Feb 01 '23 06:02 alixander

Thanks @alixander I have played with this and large diagrams like above that previously could have too much space, now have a lot less and look a lot better!

judepayne avatar Feb 01 '23 13:02 judepayne

Hi @alixander I closed this but just experimenting with generating random graphs and here's another one that seems to create a lot more empty space than strictly required:

bank: { Funds: { app8167: Data Source app8169: Flame minor app8171: Trade propel } Risk: { app8164: Credit guard } Fixed Income: { app8166: Data Source app8170: Data Source } Corporate: { app8168: Risk Sheet app8172: Apac Ace } Equities: { app8165: Book down } Finance: { app8173: Zone out } } bank.Corporate.app8168 -> bank.Corporate.app8168: client master bank.Risk.app8164 -> bank.Corporate.app8172: allocations bank.Risk.app8164 -> bank.Funds.app8169: instructions bank.Risk.app8164 -> bank.Corporate.app8168: trades bank.Funds.app8171 -> bank.Fixed Income.app8166: instructions bank.Risk.app8164 -> bank.Risk.app8164: client master bank.Funds.app8169 -> bank.Funds.app8167: instructions bank.Fixed Income.app8170 -> bank.Corporate.app8172: positions bank.Fixed Income.app8170 -> bank.Funds.app8167: security reference bank.Fixed Income.app8170 -> bank.Fixed Income.app8166: instructions bank.Funds.app8171 -> bank.Equities.app8165: positions bank.Funds.app8171 -> bank.Funds.app8171: positions

judepayne avatar Feb 01 '23 20:02 judepayne

... here's another one

as a playground link: https://play.d2lang.com/?script=jFNBTsMwELznFfMBohZKU3xAKoVKXCknbsZZwGpiG8cRINS_I6chdYJbcrO8O7OzM_YzV1uG7wRY1yqv9keAG7OYzjOGW-44No3soHDFsC54SSil0vZQyKYMj5bnBGO1oSIBdgnwIKttn3nGsLKUS4fXmtu87VvLT8pxr4Quqd8_P6IkmwwLnmelrdGWuwHJgjVKsHkjcgHHOcPScIFlR3D3XksnaWDHJcON1lvk-kN1ihVXoj8nu2B40oqga9e07RLvctqpSls5OLtGvMIgCknKoeSVI7vHe-1tw-wY1O_Ci0IL7j9LdQLYxJ12cUr_22rxHyqi1Pm8W0xAmk0Po4JY0y7PkRPDy6gtvUWi62WxYX9FZZNTrhpdybHwwfCKRG2l-4KlF7KkBI3hGGla3PLfB5x2L3eoP55UeBtifgIAAP__&layout=tala&

gavin-ts avatar Feb 01 '23 20:02 gavin-ts

@gavin-ts i think our parameters just considers this okay. e.g. grid cell is large because of the containers. i suspect if we just lower the threshold it'll be okay

alixander avatar Feb 01 '23 22:02 alixander