Skip to content

Eliminate use of no-display#132

Merged
nicolaspayette merged 6 commits intomasterfrom
eliminate-no-display
Nov 6, 2015
Merged

Eliminate use of no-display#132
nicolaspayette merged 6 commits intomasterfrom
eliminate-no-display

Conversation

@nicolaspayette
Copy link
Copy Markdown
Member

@SethTisue points out in #131 (comment) that the use of no-display should be avoided in the Models Library.

As 9437660 reveals, it is currently used in the following (curricular) models:

[info] - Forbidden primitives are not used *** FAILED ***
[info]   "/media/data/Dropbox (SESP)/w/nl5/models/HubNet Activities/Unverified/Oil Cartel Alternate HubNet.nlogo"
[info]     uses no-display
[info]   "/media/data/Dropbox (SESP)/w/nl5/models/Curricular Models/ProbLab/4 Block Stalagmites.nlogo"
[info]     uses no-display
[info]   "/media/data/Dropbox (SESP)/w/nl5/models/Curricular Models/ProbLab/4 Block Two Stalagmites.nlogo"
[info]     uses no-display (TestModels.scala:21)

I'll see if I can weed it out.

@nicolaspayette nicolaspayette self-assigned this Sep 21, 2015
@nicolaspayette nicolaspayette added this to the hexy milestone Sep 29, 2015
...and get rid of that confusing (to say the least) "super-go"
procedure.
and also get rid of `without-interruption` which I can only
assume was a leftover from an old version of the model...
nicolaspayette added a commit that referenced this pull request Nov 6, 2015
@nicolaspayette nicolaspayette merged commit a37a6f1 into master Nov 6, 2015
@nicolaspayette nicolaspayette deleted the eliminate-no-display branch November 6, 2015 21:10
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.

1 participant