Nikea / VTTools

Library that has utility tools for VisTrails
Other
1 stars 8 forks source link

Refactored VisTrails stuff from nsls2.fitting into VTTools #32

Closed ericdill closed 10 years ago

ericdill commented 10 years ago

cc @licode @tacaswell also see Nikea/NSLS2#188

coveralls commented 10 years ago

Coverage Status

Changes Unknown when pulling 35a44c201a96d5e80b350a20d0abfabb5d9c4a36 on ericdill:fit_refactor into \ on Nikea:master**.

ericdill commented 10 years ago

The python 3 failure is because of vistrails, not our library, so this is ready for review, @Nikea/development-team

tacaswell commented 10 years ago

I would just remove py3k from the travis config for this repo until VT catches up.

ericdill commented 10 years ago

ah, good call

coveralls commented 10 years ago

Coverage Status

Changes Unknown when pulling 55599d2abb7d7bb79279955f5579f723b5302d29 on ericdill:fit_refactor into \ on Nikea:master**.

ericdill commented 10 years ago

Ok, so project names... Anyone have very strong preferences on any of these? basic instructions for spinning up a scikit project

ericdill commented 10 years ago

@Nikea/development-team see previous comment

licode commented 10 years ago

scikit-xray looks good to me.

tacaswell commented 10 years ago

I like scikit-xray for the name and skxray for the import target which follows the pattern used by scikit-image and scikit-learn.

sameera2004 commented 10 years ago

I like the name scikit-xray too

ericdill commented 10 years ago

Bringing @stuwilkins and @jhill14 into the loop. What do you guys think?

ericdill commented 10 years ago

wait why is this discussion on vttools...

celiafish commented 10 years ago

Sorry for the late reply. I like that too.

Regards, Wei


Jacques grows, Jacob follows.

On Nov 11, 2014, at 1:44 PM, Sameera Abeykoon notifications@github.com wrote:

I like the name scikit-xray too

— Reply to this email directly or view it on GitHub https://github.com/Nikea/VTTools/pull/32#issuecomment-62595449.

ericdill commented 10 years ago

@tacaswell updated this branch with the skimage rename. our imports work, but numpy do not

coveralls commented 10 years ago

Coverage Status

Changes Unknown when pulling 9de81d39806e07cc88562d455aa2486678a6ffd4 on ericdill:fit_refactor into \ on Nikea:master**.