-
Notifications
You must be signed in to change notification settings - Fork 14
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
Results_Engine: Checking ToString properties instead of IComparable for ResultFiltering #2774
Results_Engine: Checking ToString properties instead of IComparable for ResultFiltering #2774
Conversation
@BHoMBot check compliance |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 39 requests in the queue ahead of you. |
@BHoMBot check required |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 29 requests in the queue ahead of you. |
The check |
The check |
@BHoMBot check required |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 13 requests in the queue ahead of you. |
The check |
The check |
@IsakNaslundBh just to let you know, I have provided a |
@IsakNaslundBh just to let you know, I have provided a |
The check |
@BHoMBot check ready-to-merge |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 18 requests in the queue ahead of you. |
NOTE: Depends on
BHoM/BHoM#1376
Issues addressed by this PR
Filtering strings rather than by IComparable to improve UX.
Test files
https://burohappold.sharepoint.com/:f:/r/sites/BHoM/02_Current/12_Scripts/02_Pull%20Request/BHoM/BHoM/Analytical_oM/%231376-ChangeResultFiltersToStringHashSets?csf=1&web=1&e=ZxWpLB
Changelog
Additional comments