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

ENH: For set outputs ignore the order #182

Open
bsipocz opened this issue Jun 1, 2022 · 2 comments
Open

ENH: For set outputs ignore the order #182

bsipocz opened this issue Jun 1, 2022 · 2 comments

Comments

@bsipocz
Copy link
Member

bsipocz commented Jun 1, 2022

It would be nice to have set comparison as opposed to adding +IGNORE_OUTPUT for lines like this one where the order in the output is not fixed:

305   >>> res = registry.search(ivoid="ivo://org.gavo.dc/flashheros/q/ssa")[0]
306   >>> res.access_modes()
Expected:
    {'ssa', 'tap#aux', 'datalink#links-1.0', 'web', 'soda#sync-1.0'}
Got:
    {'web', 'ssa', 'soda#sync-1.0', 'tap#aux', 'datalink#links-1.0'}
``
@pllim
Copy link
Contributor

pllim commented Jun 1, 2022

Maybe access_modes() should return a sorted list? It would be confusing to me as user if the order is non-deterministic.

@bsipocz
Copy link
Member Author

bsipocz commented Jun 1, 2022

yeah, I'll cross open a pyvo issue. Still, this maybe a useful feature for doctestplus.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants