flatironinstitute / mountainsort5

MountainSort spike sorting algorithm, version 5
Apache License 2.0
34 stars 8 forks source link

Add cron for trying to build on Mac and Windows #15

Closed zm711 closed 11 months ago

zm711 commented 11 months ago

@magland

I saw the issue with the mac build. This is just a draft action to build and test mountainsort5 on mac , windows and linux. If you want the cron to just build and see if it can successfully build the test part can be deleted. (I don't think GA test on M chips for macs, but users can also try to build the packages with rosetta if needed).

codecov-commenter commented 11 months ago

Welcome to Codecov :tada:

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered :open_umbrella:

JoeZiminski commented 11 months ago

Hey @zm711 this looks good! It will be interesting to see how this runs now, this PR on isosplit6 builds wheels for macos and uploads to pypi so hopefully these tests will pass on mac now, as (at least the problem I had) was being unable to install isosplit6 dependency.

zm711 commented 11 months ago

Cool @JoeZiminski. If we need to I can switch this to on push and cron to do a test run to see if things are working. Otherwise, the job won't run until this is merged. I've been having some mac dependency issues trying to work on SpikeInterface from my mac, so any patches to the infrastructure are much appreciated. :)

magland commented 11 months ago

Merging.

zm711 commented 11 months ago

Thanks @magland! We'll see what happens on Sunday when it runs!