Repository navigation
tagging for other stuff - #356
Conversation
| return self.split.repeats, self.split.folds, self.split.samples | ||
|
|
||
| def push_tag(self, tag): | ||
| """Annotates this flow with a tag on the server. |
There was a problem hiding this comment.
Should be `Annotates thi task``, right?
| Parameters | ||
| ---------- | ||
| tag : str | ||
| Tag to attach to the flow. |
| Tag to attach to the flow. | ||
| """ | ||
| data = {'flow_id': self.flow_id, 'tag': tag} | ||
| _perform_api_call("/flow/tag", data=data) |
| tag : str | ||
| Tag to attach to the flow. | ||
| """ | ||
| data = {'flow_id': self.flow_id, 'tag': tag} |
| _perform_api_call("/flow/tag", data=data) | ||
|
|
||
| def remove_tag(self, tag): | ||
| """Removes a tag from this flow on the server. |
| Parameters | ||
| ---------- | ||
| tag : str | ||
| Tag to attach to the flow. |
| tag : str | ||
| Tag to attach to the flow. | ||
| """ | ||
| data = {'flow_id': self.flow_id, 'tag': tag} |
| Tag to attach to the flow. | ||
| """ | ||
| data = {'flow_id': self.flow_id, 'tag': tag} | ||
| _perform_api_call("/flow/untag", data=data) |
There was a problem hiding this comment.
looks like I missed one search and replace.
| self.assertEqual(len(subflow_3.components), 0) | ||
|
|
||
| def test_tagging(self): | ||
| flow = openml.flows.get_flow(4024) |
There was a problem hiding this comment.
That's dangerous. If the server is purged, this flow will no longer exist. Maybe you should just query the server for a flow and take the first.
There was a problem hiding this comment.
This flow is used in another test, which is why I selected it ;) We can have the setup query for a flow and then use that everywhere, though.
There was a problem hiding this comment.
Though the server was purged and it didn't fail, right?
There was a problem hiding this comment.
But that flow is loaded from the production server.
There was a problem hiding this comment.
Querying for a flow would be better.
There was a problem hiding this comment.
lol it's actually using the production server in the other test...
|
should be good now |
2 participants
No description provided.