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

Naming scheme for new custom classifications #1

Closed
nheeren opened this issue Sep 5, 2018 · 1 comment
Closed

Naming scheme for new custom classifications #1

nheeren opened this issue Sep 5, 2018 · 1 comment
Assignees
Labels
question Further information is requested

Comments

@nheeren
Copy link
Member

nheeren commented Sep 5, 2018

@stefanpauliuk I am currently writing the function that determines the name of new custom classifications. During our call we said we would use a concatenation of the dataset name and the aspect name. An example: 1_F_steel_SankeyFlows_2008_Global_origin_process.

However, I suggest to i. reverse the order, because it is makes it clearer what kind of classification this is supposed to be, ii. use two underscores to distinguish the names from another. So the example would become: origin_process__1_F_steel_SankeyFlows_2008_Global.

Was wondering if there might be a reason not to do so...?

@nheeren nheeren added the question Further information is requested label Sep 5, 2018
@stefanpauliuk
Copy link
Member

Good idea, OK!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants