Cross Platform Compatibility - Hard coded directory separators

[storm255]

Hi.

Just downloaded it to have a poke around as it looks very impressive on the demo video for rapid development.

I might have done something wrong, but I had to do a quick find and replace on the directory string separators to get it up and running on Linux. Haven't investigated further but it might be that os.path.join() could be used for future portability.

Python 3.5.2 PyQt5 5.6.1 OpenCV 3.1.0-3

Great work. Looking forward to exploring.

[apockill]

That's very interesting, I hadn't thought of that problem. Does your solution work?

On Thu, Aug 11, 2016 at 3:18 PM storm255 notifications@github.com wrote:

Hi.

Just downloaded it to have a poke around as it looks very impressive on the demo video for rapid development.

I might have done something wrong, but I had to do a quick find and replace on the directory string separators to get it up and running on Linux. Haven't investigated further but it might be that os.path.join() could be used for future portability.

Python 3.5.2 PyQt5 5.6.1 OpenCV 3.1.0-3

Great work. Looking forward to exploring.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/apockill/uArmCreatorStudio/issues/5, or mute the thread https://github.com/notifications/unsubscribe-auth/ABqOfKfUzNK7nBHZcwscc1z5ivcr2PWTks5qeszXgaJpZM4Jh2wB .

[storm255]

It worked to get it up and running in my environment just doing a simple find and replace on the double backward slashes with a single forward slash (I think there were 12 replacements), but to be cross platform I think it would just need the

resourcesLoc = 'Resources' settings_txt = os.path.join(resourcesLoc, 'Settings.txt')

in the appropriate places.

I can do a pull request later if that would help, but I am on a project for the next few weeks.

Mark.

[apockill]

Thank you, I'm glad to hear that worked. I'll get right on that.

On Thu, Aug 11, 2016 at 5:51 PM storm255 notifications@github.com wrote:

It worked to get it up and running in my environment just doing a simple find and replace on the double backward slashes with a single forward slash (I think there were 12 replacements), but to be cross platform I think it would just need the

resourcesLoc = 'Resources' settings_txt = os.path.join(resourcesLoc, 'Settings.txt')

in the appropriate places.

I can do a pull request later if that would help, but I am on a project for the next few weeks.

Mark.

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/apockill/uArmCreatorStudio/issues/5#issuecomment-239117885, or mute the thread https://github.com/notifications/unsubscribe-auth/ABqOfPcPR--Q6vXJPXf-8bz40-9e1eiqks5qevCMgaJpZM4Jh2wB .

[apockill]

Okay, so I combed through the project and got rid of every filename concatenation, then thoroughly tested it. If you could run it and tell me if I missed anything, I'd be quite grateful!

Also, were there any steps I missed in the build instructions on my readme? Thanks!

[storm255]

All good from my perspective. Like I said, I haven't done anything with it yet, but it opens successfully here on Linux. I already had the requirements installed so no problem from that point of view as well.

Found the cameras without problem. Will have to wait till I get home to hook up to some hardware and test a few things.

Thanks for being so responsive. Have a great day.

[apockill]

If you get the chance, you can try the object recognition and tracking without needing any hardware attached. Just click "Resources", "New Vision Object" and follow the steps.

It's fun to experiment with the tracking