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

Ruletest 12-2 #1436

Merged
merged 4 commits into from
Aug 23, 2024
Merged

Ruletest 12-2 #1436

merged 4 commits into from
Aug 23, 2024

Conversation

jugonzal07
Copy link
Contributor

First cut at rule test 12-2 JSON

Copy link
Collaborator

@Jiarongx-Xie Jiarongx-Xie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think test case b should return undetermined (corresponding to Case 4 in the RDS) instead of fail. Could you double-check? Thanks @jugonzal07

@jugonzal07
Copy link
Contributor Author

I think test case b should return undetermined (corresponding to Case 4 in the RDS) instead of fail. Could you double-check? Thanks @jugonzal07

Addressed! I made the baseline case's EFLH < proposed cases EFLH. I think that should address this. I would argue that we should update the RDS Case 2 to specify that this should be such that eflh_difference < 0 instead of !=0. If it is > 0, this description would meet the requirements of Case 4.

@Jiarongx-Xie
Copy link
Collaborator

I think test case b should return undetermined (corresponding to Case 4 in the RDS) instead of fail. Could you double-check? Thanks @jugonzal07

Addressed! I made the baseline case's EFLH < proposed cases EFLH. I think that should address this. I would argue that we should update the RDS Case 2 to specify that this should be such that eflh_difference < 0 instead of !=0. If it is > 0, this description would meet the requirements of Case 4.

Thanks Juan. I further looked into the RDS. I think instead of changing Case 2 condition, we need to add not auto_receptacle_controls_b to Case 4 condition. Please let me know your thoughts. @jugonzal07 @weilixu

@weilixu weilixu changed the base branch from develop to feature/section12 August 20, 2024 17:53
@Jiarongx-Xie
Copy link
Collaborator

I changed the logic and now test case d failed where it is supposed to fail (Case 2 in RDS) instead of returning UNDETERMINED (Case 4 in RDS). Could you double-check? @jugonzal07

…ntrols to correctly trigger the undetermined case.
@Jiarongx-Xie
Copy link
Collaborator

Thank you for making the change. @jugonzal07 Everything looks good to me now.

@weilixu weilixu merged commit e4975e9 into feature/section12 Aug 23, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants