search

GollyGang / ready

A cross-platform implementation of various reaction-diffusion systems and PDEs.
GNU General Public License v3.0
766 stars 60 forks source link

Add integration test - call rdy to load, iterate and save a pattern #65

Closed timhutton closed 4 years ago

danwills commented 4 years ago

Since you merged this does that mean you can has write bash? or did you give up but there were still some changes worth merging? Very entertaining stream of comments btw! (genuine lulz) :)

I can't tell whether you got it to work from the comments above. No probs either way, just curious.

timhutton commented 4 years ago

I meant to squash merge those comments, so was a bit relaxed with them. But then I did the wrong kind of merge and now all those comments are in our master branch history. :(

It did work in the end, on Linux and macOS. Not yet on Windows, because of DLL loading issues that are hard to debug.

danwills commented 4 years ago

Heyy Tim, no dramas mate, good to have a record of the journey! Glad you got 2 platforms sorted too!

Windows platform build, eventually, surely! :D (I know you/we'll be able to work it out in the end!)

On Thu, Jan 9, 2020 at 10:08 PM Tim Hutton notifications@github.com wrote:

I meant to squash merge those comments, so was a bit relaxed with them. But then I did the wrong kind of merge and now all those comments are in our master branch history. :(

It did work in the end, on Linux and macOS. Not yet on Windows, because of DLL loading issues that are hard to debug.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/GollyGang/ready/pull/65?email_source=notifications&email_token=AAEDSJS6UDDLDRUP6EZ3E53Q44EFDA5CNFSM4KCD6VB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIQAJGI#issuecomment-572523673, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEDSJXM5O3Z3OMTMQEL5Q3Q44EFDANCNFSM4KCD6VBQ .