-
Notifications
You must be signed in to change notification settings - Fork 17
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
OSA Schema #583
OSA Schema #583
Conversation
nllong
commented
Jul 14, 2020
- Add an OSA JSON schema. The location is server/app/lib/analysis_library/schema/osa.json but I am thinking we should place in /server/schemas.
- Should probably make a datapoint.json schema as well.
@@ -123,4 +123,6 @@ group :development, :test do | |||
gem 'factory_bot_rails', '4.11.1' # 5.0.2 require ruby 2.3 | |||
gem 'mongoid-rspec' | |||
gem 'rspec-html-matchers', '0.8.1' | |||
|
|||
gem 'json-schema', '2.8.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only needed for development.
"default": 2, | ||
"minimum": 1 | ||
}, | ||
"popSize": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is popsize and popSize different?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
they world be since json is case sensitive.
"default": 2, | ||
"minimum": 0 | ||
}, | ||
"ppower": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this used?
"type": "number", | ||
"default": 5 | ||
}, | ||
"xoverdistidx": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we leave these for now even if they aren't used.
"type": "number", | ||
"default": 2 | ||
}, | ||
"toursize": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this used?
"minkowski", "maximum", "euclidean", "binary", "manhattan" | ||
] | ||
}, | ||
"normtype": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this used?
"type": "number", | ||
"default": 10 | ||
}, | ||
"mudistidx": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added algorithm params. there's some conflicts and some added before look unused?