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

Manual installation fails in mkwrapper.py #17

Open
ghost opened this issue Aug 15, 2014 · 6 comments
Open

Manual installation fails in mkwrapper.py #17

ghost opened this issue Aug 15, 2014 · 6 comments

Comments

@ghost
Copy link

ghost commented Aug 15, 2014

Dear all, I would like to make you aware that the manual installation as described in ReadMe.rst fails because mkwrapper throws an exception in line 837. The problem occurs while the wrapper tries to determine the size of one function parameter that is defined in the cspice header SpiceZpr.h. The function is "void edterm_c(...)". The problem is that the last input parameter is "SpiceDouble termpts [ ][3]" and the wrapper tries to identify the size of this array. However, because the first bracket is empty, instead of an integer, the wrapper gets an empty string. With the empty string, the wrapper cannot continue to work and throws the exception.

I tried to run everything on Windows and on Linux. The same error occurs in both cases.
I hope that I can add to the progress of this great project with this comment. I would really like to see SPICE for Python.

Cheers

@michaelaye
Copy link
Collaborator

Thanks. As already said in #16 (which I closed now for cleanliness and keeping this open), this happens for the toolkit N0065 currently. In #16 you will find a link to toolkit N0064 which works. Thanks for the patience.

@ghost
Copy link
Author

ghost commented Aug 18, 2014

Hi Michael,
thanks for your response. I checked the cspice version provided via dropbox link in #16 and noticed that it is for unix (based on the file makeall.csh). I am using Windows 7. I used the dropbox version of cspice and replaced makeall.csh with makeall.bat from the newest cspice version. Following the manual installation procedure and after running the first command, I now get the following error message:

"
File "mkwrapper.py", line 1077, in
print main(cspice_toolkit)

File "mkwrapper.py", line 1023, in main
gen_wrapper(curr_prototype, buffer)

File "mkwrapper.py", line 537, in gen_wrapper
doc = get_doc(prototype_obj.function_name)

File "mkwrapper.py", line 193, in get_doc
f.seek(f.tell()-input_len)
IOError: [Errno 22] Invalid argument
"

Thanks for your work on this project!

@michaelaye
Copy link
Collaborator

Sorry, don't have a Windows machine for debugging. I'm quite annoyed that NAIF does not seem to offer links to older versions, maybe something changed in between N64 and N65 there as well?
Hopefully somebody else can help?
Also I will be out of network for the next 2 weeks of my holidays.

@drbitboy
Copy link
Contributor

Old versions:

ftp://naif.jpl.nasa.gov/pub/naif/misc/

?

@michaelaye
Copy link
Collaborator

Ah, thanks! Didn't know about that one. The website does not mention anything about it, at least I could not find it.

@drbitboy
Copy link
Contributor

A quick fix for SPICE toolkit N0065

https://github.com/drbitboy/PySPICE

Basically just excludes a few new functions that trip up mkwrapper.py.

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

No branches or pull requests

2 participants