Skip to content

Conversation

@algor1th
Copy link
Collaborator

No description provided.

@algor1th algor1th linked an issue Aug 14, 2020 that may be closed by this pull request
5 tasks
algor1th and others added 3 commits August 14, 2020 18:14
* actor id no longer hard coded
* generate UUIDs instead of having them statically
…d. Removed some commented out debug messages.
this.addWs(ctx);
mLogger.message("Connected to Charamel VuppetMaster");
ctx.send(Strings.launchString);
// 17.8.2020 PG - deactivated - Do we need a launch string? ctx.send(Strings.launchString);
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, it is useful for testing. No point for it beyond that

Copy link
Collaborator Author

@algor1th algor1th left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the null pointer case

public void launch() {
mLogger.message("Loading Charamel VuppetMaster Executor (WebSocket) ...");
final int port = Integer.parseInt(Objects.requireNonNull(mConfig.getProperty("port")));
final String sceneflowVar =mConfig.getProperty("sceneflowVar");
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May be null

mLogger.message("Connected to Charamel VuppetMaster");

// let sceneflow know that a client has connected
if (mProject.hasVariable(sceneflowVar)) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

scenefloVar may be null, which leads to a crash here

stomack and others added 30 commits December 8, 2022 17:40
…option in cases where the REQUEST will lead to multiple instances of radio button inputs. Also contains corresponding updates to the studymaster-web example project where all the REQUEST fields have been doubled to verify and record the functioning of multiple input instances of the same type.
…ight teh right selected option in cases where the REQUEST will lead to multiple instances of radio button inputs.
…empty upon starting and ending the studymaster session i.e. when the play and stop button are pressed in VSM.
# Conflicts:
#	plugins/studymaster-web/ExampleProject/studymaster-web/sceneflow.xml
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.

Plugin for charamel via websocket

10 participants