Polydisperse Spheres - Githubissues

kartikye commented 3 years ago

Hi!

Did you by any chance implement support for spheres of different sizes?

sdatkinson commented 3 years ago

Hi.

So this version doesn't support polydisperse spheres, but it's certainly possible.

I don't have plans to implement it myself since I don't really work on the sorts of problems this solves anymore, but if you're interested in taking a crack at it the changes would basically involve

taking the diameter property in the packing class, moving it to be a property of each sphere (probably want to use radii since IIRC most calculations involve the sum of two radii which just happens to be a diameter for monodisperse).
Modify references to it accordingly (probably when making constraints & maintaining the near-neighbor lists)
Then provide the radii somehow e.g. as another column after the coordinates or as another file.

kartikye commented 3 years ago

Got it, Thanks!

Secondly, do you have a licence in mind for the code?

sdatkinson commented 3 years ago

Secondly, do you have a licence in mind for the code?

Right, apologies as this was made public quite a while back...Fixed :)

kartikye commented 3 years ago

Thank you!

On Wed, Mar 24, 2021, 4:37 PM Steven Atkinson @.***> wrote:

Secondly, do you have a licence in mind for the code?

Right, apologies as this was made public quite a while back...Fixed :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/sdatkinson/TJ/issues/1#issuecomment-806251835, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZDMP3RJTAOKE7LPMK2XEDTFJZTDANCNFSM4ZLQFCTQ .

sdatkinson / TJ

Polydisperse Spheres #1