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

'.py' extension on script functions not working #103

Closed
ghost opened this issue Nov 29, 2016 · 4 comments
Closed

'.py' extension on script functions not working #103

ghost opened this issue Nov 29, 2016 · 4 comments
Labels
Milestone

Comments

@ghost
Copy link

@ghost ghost commented Nov 29, 2016

>>> import vcs
>>> a=vcs.init()
>>> b=a.getboxfill()
>>> b.script('filename.py')
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-30-89bbd0e0a46d> in <module>()
----> 1 b.script('filename.py')

/Users/brown308/anaconda/envs/2.8/lib/python2.7/site-packages/vcs/boxfill.pyc in script(self, script_filename, mode)
   1097             fp.write("%s.ext_1 = '%s'\n" % (unique_name, self.ext_1))
   1098             fp.write("%s.ext_2 = '%s'\n" % (unique_name, self.ext_2))
-> 1099             fp.write("%s.missing = %g\n" % (unique_name, self.missing))
   1100             fp.write(
   1101                 "%s.datawc_calendar = %g\n" %

TypeError: float argument required, not tuple

@doutriaux1 Didn't this used to work?

@chaosphere2112
Copy link
Contributor

@chaosphere2112 chaosphere2112 commented Nov 30, 2016

@doutriaux1 Once again the color changeover strikes fear into the hearts and minds of our populace!

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented Nov 30, 2016

yes I know I sent a patch to @embrown the other day.

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented Nov 30, 2016

just call me Attila

@ghost
Copy link
Author

@ghost ghost commented Jan 9, 2017

@doutriaux1 @chaosphere2112 I'm pretty sure I got python output working correctly here. Whenever that's merged, we should be good to close this issue.

@ghost ghost self-assigned this Jan 9, 2017
@doutriaux1 doutriaux1 modified the milestone: 2.10 May 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants