Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Nielswps
Copy link
Contributor

No description provided.

@Nielswps Nielswps self-assigned this May 19, 2020
@Nielswps Nielswps changed the title WIP: Interface for UPPAALDriver Interface for UPPAALDriver and Singleton for management May 31, 2020
@Nielswps Nielswps marked this pull request as ready for review May 31, 2020 10:14
// Dialog title
final FileChooser filePicker = new FileChooser();
filePicker.setTitle("Choose UPPAAL server file");
filePicker.getExtensionFilters().add(new FileChooser.ExtensionFilter("UPPAAL server", "*.exe"));
Copy link
Member

Choose a reason for hiding this comment

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

on macOs/linux the server will not include *.exe

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have removed the filter, I'm unsure whether the file type should be handled in any way or just caught by exceptions when trying to run it

Copy link
Member

@yrke yrke left a comment

Choose a reason for hiding this comment

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

When uppaal server file it located, check the server and clear the warning about uppaal not found.

@yrke
Copy link
Member

yrke commented Jun 11, 2020

Also i get an "java.util.MissingResourceException: Can't find bundle for base name locale.LexicalAnalyser, locale en_US" in the background don't know if this is related.
And also having some problems with the verifyer locking up, byt maybe we should look at this in an other PR.

@Nielswps
Copy link
Contributor Author

I have removed the warning when the UPPAALDriver is set, but the interface does not correspond with this. I'm unsure how to force the interface to be updated

@yrke yrke merged commit 7c5f85f into DEIS-Tools:master Jun 15, 2020
@Nielswps Nielswps deleted the IUPPAALDriver branch June 20, 2020 19:06
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.

2 participants