WormBase / pseudoace

Modelling the WormBase ACeDB database in datomic.
4 stars 3 forks source link

pseudoace as a library #9

Closed mgrbyte closed 8 years ago

mgrbyte commented 8 years ago

@a8wright After more review of the original clojure code in the hinxton-curation repo (wb, colanade, trace), I've identified that it in fact uses more than just the pseudoace.utils module, but also uses: pseudoace.ts_import psuedoace.import

As discussed on our last hangout; It'd be good to have one name which can be used for this repository and the clojure package which it provides.

The name of the command line tool is not of any importance [1]

Since you've said you're not keen on "pseudoace" as a name, how about calling this "acedatomic"?

Proposal:

[1] Since when invoking lein run the function pointed to by :main is executed by default.

Thus, currently: lein run ace-to-dataomic is the same as: lein run

adamjohnwright commented 8 years ago

@mgrbyte I agree with your diagnosis of the situation. Feel free to change the name of the repo and the name space. It is interesting that so many other parts use the importer - not expected. Feel free to organize things so that the shared code is in one repository and included in to repositories as necessary. I will be working on datomic-to-catalyst code today. hopefully I don't run into similar complexity.

dasmoth commented 8 years ago

The curation tool depends on the importer because it supports submitting .ace-formatted patches.

mgrbyte commented 8 years ago

We decided to keep the current name pseudoace (thanks to @a8wright for agreeing!)

mgrbyte commented 8 years ago

Closed by d486d686f8d4320dee31d1f86a70002a2b134eeb