ScottishCovidResponse / SCRCIssueTracking

Central issue tracking repository for all repos in the consortium
6 stars 0 forks source link

Repository Names (and "sync tool name") #785

Closed bobturneruk closed 3 years ago

bobturneruk commented 3 years ago

GitHub organisations are often not the easiest to navigate. Having clear and systematic naming of repositories will help.

bobturneruk commented 3 years ago

I suggest the following renaming:

Old Name New Name
data_pipeline_api python_pipeline_api
SCRC-Cpp-API cpp_pipeline_api
SCRCdataAPI r_pipeline_api
JAVA-data-pipeline java_pipeline_api (also make public)
DataRegistryUtils.jl julia_pipeline_api
fdp fairdp_sync_tool
data-registry data-registry (stays same)

Incidentally, I think these are the total repositories that the data pipeline grant / team are working on.

soniamitchell commented 3 years ago

My only comment is that if the R API were ever intended to be published to CRAN, it would be preferable that the repo name match the package name, whatever that might be.

bobturneruk commented 3 years ago

Thanks @soniamitchell - I've been lazily pythonic with all the names. Is the proposed name OK for CRAN? Not sure what the rules are...

soniamitchell commented 3 years ago

Actually, I have a second comment... data-registry being hyphenated when everything else is underscored makes me uncomfortable... though if I don't look at your list I have no problem with it.

bobturneruk commented 3 years ago

I was thinking leave it be as it's OK as it is, but I agree the inconsistency is irksome!

soniamitchell commented 3 years ago

Thanks @soniamitchell - I've been lazily pythonic with all the names. Is the proposed name OK for CRAN? Not sure what the rules are...

No underscores, no hyphens, no dots, just camelcase. I meant more from a descriptive perspective though. Having "R" in the name is fine, but it likely needs some reference to "FDP", and perhaps "API" is unnecessary.

mjb3 commented 3 years ago

Hi Bob, definitely agree - DataRegistryUtils was started as a sandpit for me to learn about the DR and just spiralled from there. Can we choose something like PipelineAPI.jl instead though so that it adheres to the Julia packaging conventions? I will change the name of the package too (same reason as Sonia)

ghost commented 3 years ago

Hyphenate all the others (since they are changing anyway? Don't need to hold a modifier key to type the hyphen.

soniamitchell commented 3 years ago

Hi Bob, definitely agree - DataRegistryUtils was started as a sandpit for me to learn about the DR and just spiralled from there. Can we choose something like PipelineAPI.jl instead though so that it adheres to the Julia packaging conventions? I will change the name of the package too (same reason as Sonia)

This suits me, perhaps rFDP might suit me better though (can't have dots), and FDP.jl? or we could all have different names depending on our convention?

bobturneruk commented 3 years ago

Good idea @nathan-cummings-ukaea.

kzscisoft commented 3 years ago

Maybe I am not familiar with CRAN enough, but would CRAN care about the repository name? Surely it is the name in the properties file that is what matters.

soniamitchell commented 3 years ago

CRAN doesn't care... but it's convention.

mjb3 commented 3 years ago

Hi Bob, definitely agree - DataRegistryUtils was started as a sandpit for me to learn about the DR and just spiralled from there. Can we choose something like PipelineAPI.jl instead though so that it adheres to the Julia packaging conventions? I will change the name of the package too (same reason as Sonia)

This suits me, perhaps rFDP might suit me better though (can't have dots), or we could all have different names depending on our convention?

No worries, understood. That's just the Julia way I think. Incidentally, I also noticed this among the Julia guidelines so the point you made about FDP might be applicable here too I think: _Packages that wrap external libraries or programs should be named after those libraries or programs.

CPLEX.jl wraps the CPLEX library, which can be identified easily in a web search.

MATLAB.jl provides an interface to call the MATLAB engine from within Julia._
soniamitchell commented 3 years ago

Sorry, I kept editing my comment. How about rFDP and FDP.jl or whatever people want to call it?

mjb3 commented 3 years ago

Suits me, I'll with the flow. Wasn't it decided that we were changing FDP though because it sounds too much like FTP? Or did that get shelved?

soniamitchell commented 3 years ago

No idea, I missed the last meeting so may be out of date. Happy with whatever was decided.

ghost commented 3 years ago

I don't mind if the names don't all adhere to a common convention - as long as it is unambiguous which is which.

Currently, we have: image image image image image image

ghost commented 3 years ago

Just looking at the language tag from GitHub doesn't always help either, the actual Python API code for example is inside one of the ones tagged as C++

It's just a bit of a mess currently.

soniamitchell commented 3 years ago

I don't look at anyone else's repos so it doesn't bother me, happy to go with what's easiest for everyone else.

bobturneruk commented 3 years ago

Just looking at the language tag from GitHub doesn't always help either, the actual Python API code for example is inside one of the ones tagged as C++

It's just a bit of a mess currently.

Related https://github.com/ScottishCovidResponse/SCRCIssueTracking/issues/784

bobturneruk commented 3 years ago

Do we also need a new organisation (for the data pipeline grant outputs)?

bobturneruk commented 3 years ago

Perhaps people should propose a name that works for the language they're operating in - I agree with @nathan-cummings-ukaea - we don't need to be systematic. python_pipeline_api works for Python, I think. If we all have a name we want to use by the next time we meet, we can see if there's any major objections.

For the sync tool, if we're not using fdp or fairdp, I suggest pipeline-registry alignment tool.

richardreeve commented 3 years ago

So - we have the FAIRDataPipeline organisation, and the fair command line tool. We do still need to decide on names for:

Some of these may be complete - we just need to agree on them in the meeting on Tuesday 29th. These repos also need to be moved into the new organisation if they aren't already:

Can they be moved before the meeting on Tuesday please?

richardreeve commented 3 years ago

DPAPIs renamed to langDataPipeline.