Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

Limit possible operation tags #280

Closed
15 tasks done
forman opened this issue Jul 5, 2017 · 11 comments
Closed
15 tasks done

Limit possible operation tags #280

forman opened this issue Jul 5, 2017 · 11 comments
Assignees
Labels
Milestone

Comments

@forman
Copy link
Member

forman commented Jul 5, 2017

Expected behavior

Operations should only use tags from a limited and agreed-on set.

  • aggregate
  • anomaly
  • arithmetics
  • coregistration
  • correlation
  • index
  • io
  • outliers
  • pandas
  • plot
  • resampling
  • select
  • subset
  • timeseries
  • utility

Actual behavior

We have 20+ op tags. See dropdown list in GUI.

Specifications

0.9.0.dev3

@forman forman added the ops label Jul 5, 2017
@forman forman added this to the IPM6 milestone Jul 5, 2017
@forman forman assigned forman and JanisGailis and unassigned forman Jul 5, 2017
@JanisGailis
Copy link
Member

JanisGailis commented Jul 6, 2017

@forman, can you provide the initial input to such a set? As in, which tags it makes sense to have from user's perspective when using the GUI?

@forman
Copy link
Member Author

forman commented Jul 6, 2017

When I was young, I wrote https://github.com/CCI-Tools/cate-core/wiki/Operation-Tags

@JanisGailis
Copy link
Member

That's a good initial set. I'll use those tags and raise the question if there is an operation that doesn't fit well within that set. So, according to that list 'pearson_correlation_*' would only have the utility tag.

@forman
Copy link
Member Author

forman commented Jul 7, 2017

Plus the correlation tag, because it is its module name.

@forman
Copy link
Member Author

forman commented Jul 7, 2017

Ah, but we could add that automatically - right?

@JanisGailis
Copy link
Member

It could be done automatically, and if it's done it should be done automatically. I'm just wondering if we actually want to put module names as tags. I mean, it makes sense to have them. But then, there are currently 16 operation modules in cate/ops. Add 7 tags mentioned in the wiki and we're back to 20+.

@forman
Copy link
Member Author

forman commented Jul 12, 2017

You're right. So let's not auto-tag.

@JanisGailis
Copy link
Member

I'm going over all operations to check if the tags used are not too extensive. E.g. each operation might have a module tag (not necessarily), and maybe a tag from the list in the wiki.

@forman
Copy link
Member Author

forman commented Sep 22, 2017

@JanisGailis any news here?

@JanisGailis
Copy link
Member

@forman Moving it to paused must have been done by mistake. As far as I'm concerned, this is done. I've gone over all operations and limited the tags to what we have in wiki + (maybe) module name.

@forman
Copy link
Member Author

forman commented Sep 22, 2017

Looks ok now:

image

@forman forman modified the milestones: IPM6, 1.0 Sep 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants