Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revision of Experiments Tutorials and 'Theoretically Working' adaptations to OMF6 #5

Open
wants to merge 44 commits into
base: master
Choose a base branch
from

Conversation

mitchmusarra
Copy link

Christoph,

Experiment ruby files and tutorial files 1-3 are theoretically complete to use on the website. Filenames, experiment names, experiment output, etc has been modified to work with the changes I have made to the EDs.

Experiment tutorials 4-8 have the .draft extension as they are incomplete; I do not have enough knowledge and experience in OMF6 to accurately revise the existing tutorials myself. The experiment ruby files themselves however are theoretically working with the exception of the 'net' and '%' problems/non-implementations discussed earlier. The ruby code has simplified commented sections which should assist with future revisions.

I have also changed the filenames, eg 'TUTORIAL_01.mkd' to 'Experiment01.mkd' and 'tutorial01.rb' to 'experiment01.rb'. I guess this change will allow for better user readability.

Although the experiment tutorials and EDs will require significantly more changes as OMF evolves, these commits should serve as a solid base for these future revisions.

~Mitch

mitchmusarra and others added 30 commits June 18, 2013 23:04
Code is set out clearer and comments are slightly simplified.
Added markdown feature that highlights ruby syntax
Code is layed out nicer and easier to read and understand by a
potential experimenter + Very minor changes to actual code to prevent
WARN errors + Fixed spelling errors and sentences
Comments are more relevant to code
Fixed some description mistakes and replicated to tutorial files +
created new tutorial ruby file
+ Added references to 'ED' in comments rather than 'OML2 Application'
Changed 'g.application' to 'node.application' to work with new code
style
ERROR Object: undefined local variable or method `a' for main:Object
ERROR Object: ex3.rb:17:in `block in <top (required)>'
Correctly changed 'a' to 'app.' to associate with |app| group
+ Changed groups to |app|
Exception of 'allGroups.net.w0'
'net' option not implemented in OMF6
Changed application definition to work with OMF6
+ Addressed some errors:
14:29:01 WARN  OmfEc: [DEPRECATION] 'path=' is deprecated. Please use
'binary_path=' instead.
14:29:01 WARN  OmfEc: [DEPRECATION] 'version' is deprecated and not
supported. Please do not use it.
14:29:01 WARN  OmfEc: [DEPRECATION] 'shortDescription=' is deprecated.
Please use 'description=' instead.
Replaced 'wait' with 'after'
Changed 'addApplication' definitions
Specified a subnet mask on IP addresses (24)
Changed to nodes 9 and 10
ERROR OmfCommon: Exception 'undefined method `call' for nil:NilClass'
ERROR Object: undefined method `addPrototype'
Prototype experiment appears to be unsuported unless definition of
'Protoype' is avaliable
ERROR OmfCommon: Exception 'undefined method `call' for nil:NilClass'
mitchmusarra and others added 14 commits June 20, 2013 16:12
otg2 errors:
ERROR	Server URI '' seems to be empty
ERROR	Error parsing server destination URI ''; failed to create stream
for this destination
Reverted to 'wait' to stop experiment.
ERROR OmfCommon: Exception 'undefined method `call' for nil:NilClass'
'NET' Option #Not implemented in OMF6 - New syntax required for
experiment to work
Method of specifying nodes and further adapting code to OMF6
ERROR OmfCommon: Exception 'undefined method `call' for nil:NilClass'
`addPrototype' Must be defined
ERROR OmfCommon: Exception 'undefined method `call' for nil:NilClass'
'node.net ' #Currently not working - new syntax required for experiment
to work
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