setPropertyValue and setPropertyValueSingle are missing the parameters in the documentation

[engbergandreas]

These functions are missing their parameters in the signature https://docs.openspaceproject.com/en/releases-v0.20.0/generated/scriptingApi/openspace.html#setpropertyvalue it seems to stem from the way they are exported in the luaLibrary

OpenSpace/src/scene/scene.cpp

Line 794 in 5ecdd67

scripting::LuaLibrary Scene::luaLibrary() {

and the corresponding inline file

OpenSpace/src/scene/scene_lua.inl

Line 336 in 5ecdd67

int propertySetValue(lua_State* L) {

Should we rewrite this so it follows other lua functions and so that the documentation is properly added?

[alexanderbock]

A rewrite would require some additional feature for codegen first. The problem for setPropertyValue is that the "value" parameter can be anything as the state is passed to the property to extract the correct value. So at the time of the function call we don't actually know what a "correct" parameter type is as it depends on the property.

For example:
openspace.setPropertyValue("Scene.Earth.Scale.Scale", 1.0) is correct since Scale is a float property but openspace.setPropertyValue("Scene.Earth.Scale.Scale", "abc") would not be.

[alexanderbock]

Adding a description for the argument into

OpenSpace/src/scene/scene.cpp

Line 801 in 5ecdd67

{},

and then having a type that can represent an "Any" type