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

ifSatisfied has value 100 for constraints where first argument is False #358

Open
AlexWan0 opened this issue Dec 14, 2022 · 7 comments
Open
Assignees

Comments

@AlexWan0
Copy link
Collaborator

AlexWan0 commented Dec 14, 2022

In each iteration of testing, I expected for only 1 of the 19 ifL constraints should have an ifSatisfied value in the verifyResult dictionary as there is only a single summation value which is True at each iteration. Right now, all ifL constraints in verifyResult have an ifSatisfied value of 100 if the constraints are satisfied and 18/19 constraints in verifyResult have an ifSatisfied value of 100 if the constraints aren't satisfied as it is including ifL constraints where the first argument is False.

The command for running test.py is in the readme in mnist-arithmetic-2.

@auszok
Copy link
Collaborator

auszok commented Dec 15, 2022

@AlexWan0 Thank you for preparing the Readme. However I work on Windows and your script is for the Unix environment. Can you please rewrite it in python so it can be executed on any operating system (there is wget package in python if you want to use it).
Regarding the command line to run the test. Will it be possible that you prepare python program that will configure this parameter inside so I cam run it in my debugger without additional configuration. Thank you.

@AlexWan0
Copy link
Collaborator Author

@auszok Hi, I've added a download_checkpoints.py file, and default arguments to the argparse statements in the beginning of test.py. python test.py should be able to be run directly without command line arguments now, but you may need to change the default for checkpoint_path.

@auszok
Copy link
Collaborator

auszok commented Dec 18, 2022

@AlexWan0 Thank you for updated scripts.
The reason for all ifSatisfied begin 100 has been that this has been the default value for situation when there was not lc with antecedent True. I change it to 0 and committed the change. This results in only single lc having 100 and the rest 18 0 for your example.
Maybe the correct value should be nan in this case, thought?

@AlexWan0
Copy link
Collaborator Author

Yes, I think we just need a way to only get the ifSatisified values with antecedent True.

@hfaghihi15
Copy link
Collaborator

hfaghihi15 commented Dec 19, 2022

Thanks, @AlexWan0 and @auszok, I think any indication whether being an additional keyword showing the number of applicable constraints or NaN value showing that there is no applicable constraint.

@auszok
Copy link
Collaborator

auszok commented Dec 19, 2022

Okay, so I return NaN when there is no if LC with antecedent True.
The value means that there are if LCs with antecedent True but none is satisfied.

@hfaghihi15
Copy link
Collaborator

Hi @auszok Could you please point to the document where we have discussed the output of the verifyresult from datanode,

@AdmiralDarius could you please add a link to your document about the verifyconstraint code in the program and a bit of description of what each output from that code represents?

It would be good to keep the issues conclusive so that people can later use this in case they have similar problems.

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

No branches or pull requests

4 participants