GETVSPVersion Missing from Python API

[Jason Welstead]

Rob,

Something quirky that I came across.  It appears the GETVSPVersion API call is missing from the Python API.  I have successfully tested the call using the default vspscript API, but can't get it to work when using the Python API.

Why do I care? This function would be useful to ensure that when I compile OpenVSP to a different version of Python than either 3.6 or 3.9 (reasons, reasons...) and bring it in to my local environment (not vsppytools), I am actually using the version of OpenVSP in the Python API that I think I am.

I looked at the API source and it looks like it is there.  Perhaps I am not looking in the wrong place.

Jason

[Rob McDonald]

Good catch.  There are likely other things that are similarly missing.  Unfortunately, I haven't been able to put together a comprehensive way to verify everything.

The Python API is based on the C++ API (wrapped with SWIG).  The C++ API is really everything in src/geom_api/VSP_Geom_API.h.

The AngelScript API is registered through src/geom/ScriptMgr.cpp.

These are manually maintained, so we're lucky that there aren't more places where they diverge.

It'll be fixed in the next release.  If you need it sooner, I can make a patch available on GitHub.

Rob

--
You received this message because you are subscribed to the Google Groups "OpenVSP" group.
To unsubscribe from this group and stop receiving emails from it, send an email to openvsp+u...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/openvsp/669293d8-424f-483b-9bf4-368af0bef7b7n%40googlegroups.com.

[Jason Welstead]

Rob,

Thanks for the note.  I actually found VSP_Geom_API.h and tried to add GetVSPVersion() just below GetVSPExePath and recompiled, but the compile failed.  I assumed I was in the wrong place.

No need to rush.  If I need it that bad then I can follow up.

Jason

You received this message because you are subscribed to a topic in the Google Groups "OpenVSP" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/openvsp/Vxuke9dT2Mc/unsubscribe.
To unsubscribe from this group and all its topics, send an email to openvsp+u...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/openvsp/CAEppYpFrnQxth9NaKOmyiOquf6M8KFr_RT3W6eB8TiRm3M84qQ%40mail.gmail.com.

[Rob McDonald]

What was the compile error?  You were on the right track.

Here is what I did.

https://github.com/ramcdona/OpenVSP/commit/fc53593544804eb67d9e1273c30750cc83b9532d

On the tip of this branch

https://github.com/ramcdona/OpenVSP/tree/Fixes

Rob

To view this discussion on the web visit https://groups.google.com/d/msgid/openvsp/CABo%2BvyCa307V4TPGyJq1jpB1m5fVUaF-i%3Dex5TZ%3D18z9PDy79A%40mail.gmail.com.