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

[WIP] Implement basic tagging system for testsystems #197

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jchodera
Copy link
Member

Addresses #190

This PR begins to implement a system where we can add keyword tags to each system to enable autodiscovery of testsystems that have useful properties, like are quick to run on travis (or too slow to run on travis).

Instead of having the initializer define the tags, it would be useful to be able to query this without having to actually construct the object, since that construction itself can be slow. I'm still trying to think of how best to tackle that.

@jchodera
Copy link
Member Author

jchodera commented Jun 5, 2017

Any thoughts on this approach from @Lnaden @andrrizzi @pgrinaway?

@andrrizzi
Copy link
Contributor

Nothing particularly smart. The only thing that comes to mind is to maintain a dictionary tuple_of_tags: test_system, but you'd still have to implement the infrastructure to cycle over all test systems if you only need to match a subset of tags that compose the key.

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

Successfully merging this pull request may close these issues.

2 participants