Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added basic camera handling #3

Open
wants to merge 5 commits into
base: V5
Choose a base branch
from
Open

added basic camera handling #3

wants to merge 5 commits into from

Conversation

mattvenn
Copy link
Collaborator

No description provided.

@gbaman
Copy link

gbaman commented Apr 13, 2014

Looking awesome! This is a quite cool feature I am now rather excited for... Taking photos... WITH SCRATCH!
Might want to merge https://github.com/mattvenn/scratch_gpio/pull/1 before merging into main scratchGPIO code

@cymplecy
Copy link
Owner

FANTASTIC - 1st collaboration on ScratchGPIO on Github

But.... Due to fact that I've

  1. Changed to using scratchgpio_hander5.py instead of scratchgpio_handler5dev.py in dev branch
  2. changed name of dev branch to 5dev
  3. Deleted old V5Dev
  4. made major changes since initial clone ..... etc etc :)

git merge wasn't sucessfull (Can't blame it really) :)

But I've manually merged and my 5dev branch now has the code in it and online to download
http://cymplecy.wordpress.com/2013/11/26/scratchgpio-development-testing/

If you don't mind discarding your git stuff and re-setting up off 5dev branch I promise to not muck around again :)

Note - I moved the code so that it doesn't need an add-on to be specified as we'd want it available to any Scratch script/add-on board

1

@cymplecy
Copy link
Owner

@gbaman Just added #1 with variable path as well - Ta :)

@cymplecy
Copy link
Owner

Done

On 13 April 2014 13:18, Andrew Mulholland [email protected] wrote:

Looking awesome! This is a quite cool feature I am now rather excited
for... Taking photos... WITH SCRATCH!
Might want to merge mattvenn#1https://github.com/mattvenn/scratch_gpio/pull/1before merging into main scratchGPIO code

Reply to this email directly or view it on GitHubhttps://github.com//pull/3#issuecomment-40306244
.

@cymplecy
Copy link
Owner

Variable path doesn't work because scratchgpio is run as sudo so is saving to /root/photos instead of user ~/photos :(

@mattvenn
Copy link
Collaborator Author

Check
http://stackoverflow.com/questions/8633461/how-to-keep-environment-variables-when-using-sudo
But use $HOME
On 14 Apr 2014 00:05, "cymplecy" [email protected] wrote:

Variable path doesn't work because scratchgpio is run as sudo so is saving
to /root/photos instead of user ~/photos :(

Reply to this email directly or view it on GitHubhttps://github.com//pull/3#issuecomment-40323042
.

@cymplecy
Copy link
Owner

I've updated 5dev branch with code to cope with sudo usuage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants