Opened 7 years ago

Closed 22 months ago

#8054 closed task (wontfix)

Remove command-line interface

Reported by: gsathya Owned by: metrics-team
Priority: Medium Milestone:
Component: Metrics/Compass Version:
Severity: Normal Keywords:
Cc: karsten, delber@… Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Now that we have a web frontend, I don't think anyone uses the cli at all. The cli printing code is a major pain if we want to extend compass since the printing code is a mess.

I suggest we entirely drop the cli code, if it isn't going to bother anyone.

Child Tickets

Change History (7)

comment:1 Changed 7 years ago by karsten

Cc: delber@… added

Can we instead add an abstraction layer between presentation (web or CLI) and application logic (filtering, grouping, etc.)? We should probably even do that when dropping either web or CLI to make the code cleaner. I'm not so sure that nobody uses the CLI. Maybe, if really nobody uses the CLI, instead of dropping it, we should fork the two projects. Having the mentioned abstraction layer would be nicer, though.

comment:2 in reply to:  1 Changed 7 years ago by gsathya

Replying to karsten:

Can we instead add an abstraction layer between presentation (web or CLI) and application logic (filtering, grouping, etc.)? We should probably even do that when dropping either web or CLI to make the code cleaner.

Yeah, the cli code is coupled with application logic now, so even if we have to delete the cli code we'll have to refactor that part. So it makes sense to create an abstraction layer then.

I'm not so sure that nobody uses the CLI. Maybe, if really nobody uses the CLI, instead of dropping it, we should fork the two projects. Having the mentioned abstraction layer would be nicer, though.

Well, I'm mostly inferring this from the lack of any bugs reported related to the CLI as compared to the web interface. It could just mean that the cli is really good but ..

comment:3 Changed 7 years ago by karsten

Here's another option: we could copy relevant parts from compass.py into app.py and optimize them towards the web application. The compass.py file would stay in place for CLI users, but wouldn't have to be updated whenever the web application is extended. Would that make development easier?

comment:4 Changed 7 years ago by delber

I would not mind if the CLI disappears. I think it's easier for anyone to run the CLI, but as long as the web application is reliable, this should not matter.

comment:5 Changed 2 years ago by karsten

Severity: Normal
Summary: remove cli code from compass?Remove command-line interface

Tweak the summary.

comment:6 Changed 2 years ago by karsten

Owner: changed from gsathya to metrics-team
Status: newassigned

I believe that gsathya has stopped working on any of these tickets quite a while ago. Reassigning to the friendly metrics-team user. (gsathya, thanks for having worked on all these tickets back when you did!)

comment:7 Changed 22 months ago by irl

Resolution: wontfix
Status: assignedclosed

The metrics team has put a goal of shutting down Compass in its roadmap and merging functionality with Relay Search (previously known as Atlas). This ticket is specific to Compass and as development on Compass has ceased, I am marking this ticket as wontfix.

See #23517 for information on the planned work to integrate Compass functionality into Relay Search.

Note: See TracTickets for help on using tickets.