-
Notifications
You must be signed in to change notification settings - Fork 1
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
Generalize .spec to allow optimization over stability radii #4
Comments
In the light of #3, another option is to get rid of the .spec entirely and let the v2-API user-code construct an equivalent of it when interacting with the server's API. That would be the approach I'd most likely chose until we come up with an alternative .spec format that would also include multiple networks to specify objectives, etc. |
During today's meeting this was agreed upon. In addition, @konstantin-korovin suggested to rename |
Clarification: |
KK: split
|
|
The current .spec is a list of per-feature annotations (see doc/spec for details):
label
: (used for variable naming as well as for feature names in the dataset when training / preparing)range
there):int
,float
or explicit list of categorical valuestype
there):knob
,input
,response
response
vars:rad-abs
orrad-rel
, each with a numeric constantsafe
values ("the grid")default
value (unused by the solver, as of now)Example for the
Timing
variable:This format is too restrictive for the optimization we decided to employ for Shai: Include a new
knob
variable labelledtrad
in the formula, of typeint
which is used to specify therad-abs
around the originalTiming
variable.trad
does not need any particular stability radius, so we can just require a counter-example'strad
to be the same as the candidate's. This is possible in our FMCAD formulation by a specific \theta(a,b):a.trad = b.trad /\ -a.trad <= a.Timing - b.Timing < a.trad
. It is not possible in the current .spec format, for two reasons:rad-abs
orrad-rel
.In any case, item 1. above is not easy to change and my suggestion is to move to a new .spec version, which might look something like this:
There are at least two options to solve item 2. above:
rad-abs
/rad-rel
part, e.g., userad-abs: "trad"
forTiming
The text was updated successfully, but these errors were encountered: