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

Remove obsolete widgets #77

Closed
ajdapretnar opened this issue Jun 30, 2017 · 10 comments
Closed

Remove obsolete widgets #77

ajdapretnar opened this issue Jun 30, 2017 · 10 comments

Comments

@ajdapretnar
Copy link
Contributor

Clean up Prototypes add-on.
Remove:

  • Last File Capture (doesn't work)
  • ModelMap Projection Rank (is implemented as Rank Projections elsewhere)
  • CreateClass (is now in core Orange)
@jerneju
Copy link
Contributor

jerneju commented Jul 3, 2017

If biolab/orange3#2445 is approved and merged, we might want to consider removing two other widgets as well:

  • Linear Projection
  • FreeViz

@PrimozGodec
Copy link
Collaborator

PrimozGodec commented Oct 27, 2017

FreeViz was updated recently so I guess we leave it in prototypes? @jerneju

@ajdapretnar
Copy link
Contributor Author

The idea is to avoid duplicate widgets (causes problems when browsing by widget name) and to keep Prototypes clean. I don't want it to be a widget junkyard. 😆 As for FreeViz, we can perhaps remove it once it is properly merged in core?

@PrimozGodec
Copy link
Collaborator

So I will leave it in prototypes for now. All other widgets from the list will be removed.

@jerneju
Copy link
Contributor

jerneju commented Oct 27, 2017

@ajdapretnar : FreeViz is already merged in core. Linear Projection is not.

@ajdapretnar
Copy link
Contributor Author

You are right, how did I miss this!? This requires a blog asap (well, after the release, obviously).
As for LinProj, I have it in core. You probably mean the updates to LinProj?

@jerneju
Copy link
Contributor

jerneju commented Oct 27, 2017

Yes, updates.

@ajdapretnar
Copy link
Contributor Author

Data Sets is also redundant.

@ajdapretnar
Copy link
Contributor Author

Then for LinProj I think it depends where you are pushing the changes? If you need it in Prototypes for testing the changes, it should stay here. If you are pushing directly to core, then I vote for removing it.

@jerneju
Copy link
Contributor

jerneju commented Oct 27, 2017

I am pushing changes directly to Core. As far as I'm concerned both FreeViz and Linear Projection can be removed from Prototypes.

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

No branches or pull requests

3 participants